You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Steve Carlin (Code Review)" <ge...@cloudera.org> on 2021/08/13 03:15:34 UTC

[Impala-ASF-CR] IMPALA-10858: HMS request for acid fds fixed.

Hello Impala Public Jenkins, 

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

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

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

Change subject: IMPALA-10858: HMS request for acid fds fixed.
......................................................................

IMPALA-10858: HMS request for acid fds fixed.

This commit addresses a problem for full acid tables. When the
file descriptors are requested on an HMS call, the insert and
delete file descriptors need to be gathered up and sent back
to the caller.

Also changed the visibility of some methods in SingleNodePlanner
and an object within HdfsPartition to allow extensibility by
a third party tool.

Change-Id: I055960bf0471d62bc37dda3403f58f86f9b5f596
---
M fe/src/main/java/org/apache/impala/catalog/CatalogHmsAPIHelper.java
M fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java
M fe/src/main/java/org/apache/impala/planner/SingleNodePlanner.java
M fe/src/test/java/org/apache/impala/catalog/metastore/CatalogHmsFileMetadataTest.java
4 files changed, 81 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/60/17760/2
-- 
To view, visit http://gerrit.cloudera.org:8080/17760
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I055960bf0471d62bc37dda3403f58f86f9b5f596
Gerrit-Change-Number: 17760
Gerrit-PatchSet: 2
Gerrit-Owner: Steve Carlin <sc...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>