[GitHub] [hive] kasakrisz commented on a diff in pull request #4431: HIVE-27448: Hive Iceberg: Merge column stats

["kasakrisz (via GitHub)" <gi...@apache.org>]

kasakrisz commented on code in PR #4431:
URL: https://github.com/apache/hive/pull/4431#discussion_r1269307040


##########
iceberg/iceberg-handler/src/test/results/positive/col_stats.q.out:
##########
@@ -96,7 +96,7 @@ avg_col_len         	3.4444444444444446
 max_col_len         	5                   
 num_trues           	                    
 num_falses          	                    
-bit_vector          	HL                  
+bit_vector          	                    

Review Comment:
   I did a quick search and `bit_vector` has `HL` in case of non iceberg tables in our q test outs when the column stats present for that column. 
   @simhadri-g Could you please look into why was it removed in this case?



-- 
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: gitbox-unsubscribe@hive.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org