kgyrtkirk commented on a change in pull request #1562:
URL: https://github.com/apache/hive/pull/1562#discussion_r512588759
##########
File path: ql/src/test/results/clientpositive/llap/sharedwork_semi.q.out
##########
@@ -541,7 +541,7 @@ STAGE PLANS:
Map Operator Tree:
TableScan
alias: s
- filterExpr: (ss_sold_date_sk is not null and
((ss_sold_date_sk BETWEEN DynamicValue(RS_7_d_d_date_sk_min) AND
DynamicValue(RS_7_d_d_date_sk_max) and in_bloom_filter(ss_sold_date_sk,
DynamicValue(RS_7_d_d_date_sk_bloom_filter))) or (ss_sold_date_sk BETWEEN
DynamicValue(RS_21_d_d_date_sk_min) AND DynamicValue(RS_21_d_d_date_sk_max) and
in_bloom_filter(ss_sold_date_sk,
DynamicValue(RS_21_d_d_date_sk_bloom_filter))))) (type: boolean)
+ filterExpr: (((ss_sold_date_sk BETWEEN
DynamicValue(RS_7_d_d_date_sk_min) AND DynamicValue(RS_7_d_d_date_sk_max) and
in_bloom_filter(ss_sold_date_sk, DynamicValue(RS_7_d_d_date_sk_bloom_filter)))
or (ss_sold_date_sk BETWEEN DynamicValue(RS_21_d_d_date_sk_min) AND
DynamicValue(RS_21_d_d_date_sk_max) and in_bloom_filter(ss_sold_date_sk,
DynamicValue(RS_21_d_d_date_sk_bloom_filter)))) and ss_sold_date_sk is not
null) (type: boolean)
Review comment:
I've tried to retain the order - which have placed the bloom related
checks at the end.
I recall that there was a ticket about ordering conditionals - but I can't
find the related ticket; do I remember incorrectly?
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
