Hello Quanlong Huang, Wenzhe Zhou, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/20505

to look at the new patch set (#3).

Change subject: IMPALA-12462: Update only changed partitiones after COMPUTE 
STATS
......................................................................

IMPALA-12462: Update only changed partitiones after COMPUTE STATS

This is mainly a revert of https://gerrit.cloudera.org/#/c/640/ but
some parts had to be updated due to changes in Impala.
See IMPALA-2201 for details about why this optimization was removed.

The patch can massively speed up COMPUTE STATS statement when the
majority of partitions has no changes.
COMPUTE STATS tpcds_parquet.store_sales;
before: 12s
after:   1s

Besides the DDL speed up the number of HMS events generated is also
reduced.

Testing:
- added test to verify COMPUTE STATS output
- correctness of cases when something is modified should be covered
  by existing tests
- core tests passed

Change-Id: If2703e0790d5c25db98ed26f26f6d96281c366a3
---
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M testdata/workloads/functional-query/queries/QueryTest/compute-stats.test
2 files changed, 35 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/05/20505/3
--
To view, visit http://gerrit.cloudera.org:8080/20505
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If2703e0790d5c25db98ed26f26f6d96281c366a3
Gerrit-Change-Number: 20505
Gerrit-PatchSet: 3
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>

Reply via email to