Github user zellerh commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1451#discussion_r169734207
--- Diff: core/sql/executor/ex_hash_grby.cpp ---
@@ -1027,8 +1027,10 @@ void
ex_hash_grby_tcb::returnResultCurrentRow(HashRow * dataPointer)
upParentEntry->upState.downIndex = parentQueue_.down->getHeadIndex();
// if stats are to be collected, collect them.
- if (bmoStats_)
- bmoStats_->incActualRowsReturned();
+ ExOperStats *statsEntry = getStatsEntry();
--- End diff --
bmoStats_ is also used for reporting memory-related counters, so we still
need it.
---
