Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17919
to look at the new patch set (#5).
Change subject: IMPALA-10975: Minor refactoring in alter table ddl op in
CatalogOpExecutor
......................................................................
IMPALA-10975: Minor refactoring in alter table ddl op in
CatalogOpExecutor
For almost all alter table DDL operations in catalogOpExecutor,
a table is added to catalog update at a common place in the end
if reloadMetadata is true. However for few sub ddl operations like ADD
and DROP partitions, the table to update catalog is performed locally.
This patch is to refactor addTableToCatalogUpdate() and call it at one
place for all the sub ddls.
After this patch, a table (after alter table ddl) is added to catalog
update if its old catalog version does not match the current catalog
version.
Testing:
Relying on existing tests since it is a minor refactoring.
Change-Id: Ifbadcab68b4599ad18b681b1284052a47b74d802
---
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
1 file changed, 36 insertions(+), 31 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/19/17919/5
--
To view, visit http://gerrit.cloudera.org:8080/17919
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ifbadcab68b4599ad18b681b1284052a47b74d802
Gerrit-Change-Number: 17919
Gerrit-PatchSet: 5
Gerrit-Owner: Sourabh Goyal <soura...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
