uber / aresdb

A GPU-powered real-time analytics storage and query engine.
https://eng.uber.com/aresdb/
Apache License 2.0
3.03k stars 234 forks source link

Useless field assignment in `aql_nonaggr_batchexecutor.go` #273

Open xiemaisi opened 5 years ago

xiemaisi commented 5 years ago

Describe the bug

It looks like this assignment to bc.size is dead (bc is a local variable which does not escape, and the field is not read anywhere).

To Reproduce

I found this issue using static analysis, so I cannot provide reproduction steps. I think this probably has no observable effect in practice.