[GitHub] trafodion pull request #1454: [TRAFODION-2969] Fix interaction of [first n] ...

[zellerh]

Github user zellerh commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1454#discussion_r171012343
  
    --- Diff: core/sql/optimizer/RelExpr.cpp ---
    @@ -11388,8 +11388,9 @@ void RelRoot::setMvBindContext(MvBindContext * 
pMvBindContext)
       pMvBindContextForScope_ = pMvBindContext;
     }
     
    -void RelRoot::addOneRowAggregates(BindWA* bindWA)
    +NABoolean RelRoot::addOneRowAggregates(BindWA* bindWA, NABoolean 
forceGroupByAgg)
     {
    +  NABoolean GroupByAggNodeAdded = FALSE;
    --- End diff --
    
    A small nit: This variable should start with a lower case letter.


---