zabetak commented on a change in pull request #1959:
URL: https://github.com/apache/hive/pull/1959#discussion_r584272249
##########
File path:
ql/src/test/results/clientpositive/perf/tpcds30tb/tez/cbo_query15.q.out
##########
@@ -15,6 +15,6 @@ HiveSortLimit(sort0=[$0], dir0=[ASC], fetch=[100])
HiveFilter(condition=[AND(IS NOT NULL($2), IS NOT NULL($33))])
HiveTableScan(table=[[default, catalog_sales]],
table:alias=[catalog_sales])
HiveProject(d_date_sk=[$0])
- HiveFilter(condition=[AND(=($6, 2000), =($10, 2))])
+ HiveFilter(condition=[AND(=($10, 2), =($6, 2000))])
Review comment:
Thanks for the info @jcamachor . @soumyakanti3578 also expressed some
concerns around these changes. Truth is that it didn't cross my mind that
condition order matters. I was suspecting that the changes were related to
https://issues.apache.org/jira/browse/CALCITE-4073 or
https://issues.apache.org/jira/browse/CALCITE-2450. We will check it out ;)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
