Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/24294 )

Change subject: IMPALA-14970: Time-travel queries can throw 
UnsupportedOperationException
......................................................................

IMPALA-14970: Time-travel queries can throw UnsupportedOperationException

Time-travel queries could throw UnsupportedOperationException when
they found files replicated on data nodes that were not in the host
index of the table. It's because ListMap's populate() method did not
copy the given list, but wrapped the original list via
Collections.synchronizedList(). When the given list was immutable
then we got the above error when we wanted to extend the host
index (during time-travel).

This patch fixes ListMap's populate() method to copy the given list.

Testing
* unit tests added for ListMap
* e2e tests added for Iceberg tables + time-travel

Change-Id: I3773eb7a37e9918501bfa8a22707967e79024aca
Reviewed-on: http://gerrit.cloudera.org:8080/24294
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M fe/src/main/java/org/apache/impala/util/ListMap.java
A fe/src/test/java/org/apache/impala/util/ListMapTest.java
M tests/query_test/test_iceberg.py
3 files changed, 142 insertions(+), 3 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/24294
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3773eb7a37e9918501bfa8a22707967e79024aca
Gerrit-Change-Number: 24294
Gerrit-PatchSet: 4
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>

Reply via email to