Hello Quanlong Huang, Manish Maheshwari, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-12131: For non-partitioned tables, defer the 
resetPartitions() until the file metadata is loaded.
......................................................................

IMPALA-12131: For non-partitioned tables, defer the resetPartitions() until the 
file metadata is loaded.

For non-partitioned tables, if the catalog update request
fails after resetting the partition, its partition map size
will be 0, which will cause the next catalog update requests
to fail due to Exception:"java.util.NoSuchElementException".
Currently, the only solution to this problem is to invalidate
the metadata of the table. For non-partitioned tables, defer
the resetPartitions() until the file metadata is loaded.

Testing:
1. Added a test which simulates a failure in a catalog update
request by throwing an exception through the debug action and
confirms that subsequent catalog update requests are not
affected by the failure.

Change-Id: I28e76a73b7905c24eb93b935124d20ea7abe8513
---
M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M fe/src/main/java/org/apache/impala/util/DebugUtils.java
M tests/query_test/test_insert.py
4 files changed, 44 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/19878/19
--
To view, visit http://gerrit.cloudera.org:8080/19878
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I28e76a73b7905c24eb93b935124d20ea7abe8513
Gerrit-Change-Number: 19878
Gerrit-PatchSet: 19
Gerrit-Owner: ttttttz <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Manish Maheshwari <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: ttttttz <[email protected]>

Reply via email to