Vihang Karajgaonkar has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16170
Change subject: IMPALA-9946: Use table id when comparing transactional state ...................................................................... IMPALA-9946: Use table id when comparing transactional state This change adds support for catalog client's to optionally provide a table id when fetching the table metadata using the GetPartialCatalogObject API. This table id is used to compare the catalog table is same as the one which is requested by client. If the table id matches, we compare the ValidWriteIdList to provide a consistent view of the metadata to the clients. Testing: Added a new test which drops and recreates the table from hive to introduce the false positive in the existing ValidWriteIdList comparison logic. After the patch, the test succeeds. Change-Id: I58c5bd58c4eb5663647c01ecd738b661e4e4cd74 --- M common/thrift/CatalogService.thrift M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java M fe/src/main/java/org/apache/impala/util/AcidUtils.java M fe/src/test/java/org/apache/impala/catalog/PartialCatalogInfoWriteIdTest.java M shaded-deps/pom.xml 5 files changed, 130 insertions(+), 8 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/70/16170/2 -- To view, visit http://gerrit.cloudera.org:8080/16170 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I58c5bd58c4eb5663647c01ecd738b661e4e4cd74 Gerrit-Change-Number: 16170 Gerrit-PatchSet: 2 Gerrit-Owner: Vihang Karajgaonkar <[email protected]>
