Github user traflm commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1444#discussion_r169511829
--- Diff: core/sql/optimizer/GroupAttr.cpp ---
@@ -1793,6 +1793,8 @@ void
GroupAttributes::resolveCharacteristicInputs(const ValueIdSet& externalInpu
ItemExpr * vidExpr = vid.getItemExpr();
if ((vidExpr->getOperatorType() == ITM_CURRENT_USER) ||
(vidExpr->getOperatorType() == ITM_CURRENT_TIMESTAMP) ||
+ (vidExpr->getOperatorType() == ITM_UNIQUE_SHORT_ID) ||
+ (vidExpr->getOperatorType() == ITM_UNIQUE_ID) ||
--- End diff --
You are right, Hans, I also think in the opposite, I just simply follow the
current_timestamp behavior, and this really is not what I originally want. So I
will change uuid() to be evaluated once per row.
---
