Mihaly Szjatinya has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22474 )

Change subject: IMPALA-11597: Unset impala.lastComputeStatsTime during DROP 
STATS
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/22474/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22474/2//COMMIT_MSG@7
PS2, Line 7: IMPA
> Is it still WIP?
No, removed


http://gerrit.cloudera.org:8080/#/c/22474/2/testdata/workloads/functional-query/queries/QueryTest/iceberg-compute-stats.test
File 
testdata/workloads/functional-query/queries/QueryTest/iceberg-compute-stats.test:

http://gerrit.cloudera.org:8080/#/c/22474/2/testdata/workloads/functional-query/queries/QueryTest/iceberg-compute-stats.test@77
PS2, Line 77: ---- QUERY
            : # Table-level stats are not affected by DROP STATS.
            : describe formatted ice_alltypes;
            : ---- RESULTS: VERIFY_IS_SUBSET
            : '','numFiles            ','1                   '
            : '','numRows             ','2                   '
            : row_
> Do you also want to resolve this for Iceberg tables in HiveCatalog?
Pls check the new patch, it works and doesn't break any other test, but there 
is a comment which makes it uncertain; 
https://github.com/apache/Impala/blob/master/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java#L2818



--
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: 3
Gerrit-Owner: Mihaly Szjatinya <msz...@pm.me>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Mihaly Szjatinya <msz...@pm.me>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>
Gerrit-Comment-Date: Thu, 06 Mar 2025 20:18:03 +0000
Gerrit-HasComments: Yes

Reply via email to