kgyrtkirk commented on a change in pull request #860: HIVE-22408 The fix for 
CALCITE-2991 creates wrong results on edge case
URL: https://github.com/apache/hive/pull/860#discussion_r356664585
 
 

 ##########
 File path: ql/src/test/results/clientpositive/llap/mapjoin_hint.q.out
 ##########
 @@ -648,7 +651,7 @@ STAGE PLANS:
                         outputColumnNames: _col0, _col1, _col2, _col3, _col4, 
_col5, _col6, _col7, _col8
                         input vertices:
                           1 Reducer 3
-                        Statistics: Num rows: 1 Data size: 816 Basic stats: 
COMPLETE Column stats: NONE
+                        Statistics: Num rows: 0 Data size: 0 Basic stats: NONE 
Column stats: NONE
 
 Review comment:
   it's great that now we are able to deduce that there will be 0 rows...
   I'm wondering if there will be any sideeffects of this `Basic stats: NONE`.

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