jcamachor commented on a change in pull request #729: HIVE-22003
URL: https://github.com/apache/hive/pull/729#discussion_r306069305
 
 

 ##########
 File path: 
ql/src/test/results/clientpositive/perf/tez/constraints/query32.q.out
 ##########
 @@ -115,7 +115,7 @@ Stage-0
                               Select Operator [SEL_114] (rows=285116600 
width=119)
                                 Output:["_col0","_col1","_col2"]
                                 Filter Operator [FIL_112] (rows=285116600 
width=119)
-                                  predicate:(cs_ext_discount_amt is not null 
and cs_sold_date_sk is not null)
+                                  predicate:(cs_ext_discount_amt is not null 
and cs_sold_date_sk is not null and cs_item_sk BETWEEN 
DynamicValue(RS_28_item_i_item_sk_min) AND 
DynamicValue(RS_28_item_i_item_sk_max) and in_bloom_filter(cs_item_sk, 
DynamicValue(RS_28_item_i_item_sk_bloom_filter)))
 
 Review comment:
   That the SJ is not there will not lead to incorrect results, but can lead to 
a performance penalty.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to