kasakrisz commented on code in PR #2855: URL: https://github.com/apache/hive/pull/2855#discussion_r890894844
########## ql/src/test/results/clientpositive/llap/check_constraint.q.out: ########## @@ -2061,65 +2061,87 @@ PREHOOK: query: explain cbo update acid_uami_n0 set de = 893.14 where de = 103.0 PREHOOK: type: QUERY PREHOOK: Input: default@acid_uami_n0 PREHOOK: Output: default@acid_uami_n0 +PREHOOK: Output: default@acid_uami_n0 POSTHOOK: query: explain cbo update acid_uami_n0 set de = 893.14 where de = 103.00 or de = 119.00 POSTHOOK: type: QUERY POSTHOOK: Input: default@acid_uami_n0 POSTHOOK: Output: default@acid_uami_n0 +POSTHOOK: Output: default@acid_uami_n0 CBO PLAN: -HiveSortExchange(distribution=[any], collation=[[0]]) - HiveProject(row__id=[$5], i=[$0], _o__c2=[893.14:DECIMAL(5, 2)], vc=[$2]) - HiveFilter(condition=[AND(IN($1, 103:DECIMAL(3, 0), 119:DECIMAL(3, 0)), enforce_constraint(IS NOT FALSE(>=(893.14, CAST($0):DECIMAL(5, 2)))))]) Review Comment: The original update statement is rewritten to multi insert statement and constraint check is not supported in CBO when inserting. It is done by physical plan. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
