trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zellerh <...@git.apache.org>
Subject [GitHub] trafodion pull request #1451: [TRAFODION-2965] Fix row count stats for parti...
Date Wed, 21 Feb 2018 18:30:38 GMT
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.


---

Mime
View raw message