Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1543#discussion_r187433575 --- Diff: core/sql/optimizer/RelExpr.cpp --- @@ -7880,11 +7880,17 @@ NABoolean GroupByAgg::tryToPullUpPredicatesInPreCodeGen( else pulledPredicates += tempPulledPreds; + // just remove pulled up predicates from char. input + ValueIdSet newInputs(getGroupAttr()->getCharacteristicInputs()); + myLocalExpr += selectionPred(); + myLocalExpr -= tempPulledPreds; + myLocalExpr.weedOutUnreferenced(newInputs); --- End diff -- I stepped through the code with your test case that reproduces the error. Looks like your fix does the right thing.
---