Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/22474 )
Change subject: WIP IMPALA-11597: DROP STATS should unset table property impala.lastComputeStatsTime ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/22474/1/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java: http://gerrit.cloudera.org:8080/#/c/22474/1/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@2824 PS1, Line 2824: boolean droppedLastCompute = : msTbl.getParameters().remove(HdfsTable.TBL_PROP_LAST_COMPUTE_STATS_TIME) != null; Looks like there is a detail missing here that is not explained. Why it matter to drop this property now? What other operation relies on value of this property? 'impala.lastComputeStatsTime' should be informational only and not critical, but maybe something changed recently that makes this important. -- To view, visit http://gerrit.cloudera.org:8080/22474 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id743bc9779141df7a26a95a0ea201ef6fc6aeff9 Gerrit-Change-Number: 22474 Gerrit-PatchSet: 1 Gerrit-Owner: Mihaly Szjatinya <msz...@pm.me> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Comment-Date: Wed, 12 Feb 2025 17:14:34 +0000 Gerrit-HasComments: Yes
