[email protected] has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21087


Change subject: IMPALA-12851: Fix AllocWriteIdEvent process issue to add 
txnId-tableWriteIds mapping
......................................................................

IMPALA-12851: Fix AllocWriteIdEvent process issue to add txnId-tableWriteIds 
mapping

During AllocWriteIdEvent process, txnId to tableWriteIds mapping is
not added to catalog in the following cases:
1. When CREATE_TABLE event is followed by ALLOC_WRITE_ID_EVENT for
table in the same batch of MetastoreEventsProcessor.processEvents(),
process AllocWriteIdEvent do not add txnId to tableWriteIds mapping.
2. When table is not loaded.

This patch removes the usage of get table from catalog in the
event constructors. Also adds the txnId to tableWriteIds mapping in
catalog even when table is not loaded. And ensures the write ids
are not added to table if it is a non-partitioned table.

Testing:
- Added tests in MetastoreEventsProcessorTest

Change-Id: I8b1a918befd4ee694880fd4e3cc04cb55b64955f
---
M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java
M fe/src/main/java/org/apache/impala/catalog/TableWriteId.java
M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
M 
fe/src/test/java/org/apache/impala/catalog/events/MetastoreEventsProcessorTest.java
4 files changed, 110 insertions(+), 39 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/87/21087/1
--
To view, visit http://gerrit.cloudera.org:8080/21087
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8b1a918befd4ee694880fd4e3cc04cb55b64955f
Gerrit-Change-Number: 21087
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <[email protected]>

Reply via email to