deniskuzZ commented on code in PR #5648:
URL: https://github.com/apache/hive/pull/5648#discussion_r2003624978


##########
iceberg/iceberg-handler/src/test/results/positive/delete_iceberg_copy_on_write_partitioned.q.out:
##########
@@ -121,140 +121,140 @@ STAGE PLANS:
             Map Operator Tree:
                 TableScan
                   alias: tbl_ice
-                  Statistics: Num rows: 6 Data size: 582 Basic stats: COMPLETE 
Column stats: COMPLETE
+                  Statistics: Num rows: 6 Data size: 588 Basic stats: COMPLETE 
Column stats: PARTIAL

Review Comment:
   After the update/delete Hive doesn't recompute stats automatically and on 
the next insert it only writes stats generated by that insert since prev stats 
is not accurate/not exists. 
   However, before we had a bug and  did a fallback to HMS and used whatever 
was previously stored there (no snapshot id validation, similar to ACID 
writeIds) and returned marker as COMPLETE.



-- 
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-unsubscr...@hive.apache.org

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

Reply via email to