Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/20523 )
Change subject: IMPALA-12477: Make Iceberg planFiles() use multiple threads ...................................................................... Patch Set 5: Code-Review-1 I looked at the implementation of Java's Iterable.forEach() and I couldn't figure out why ParallelIterator works correctly with forEach(), but not correctly with a for-loop. So I did more exercises, collected more jstacks, and it turned out they really don't differ, I just had bad luck when I first investigated the issue, i.e. the initial Jstacks seemed to back my hypotesis, but it is actually wrong. -- To view, visit http://gerrit.cloudera.org:8080/20523 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I00db941dd5ac9917cd91d990fccf37e5bcfddbfc Gerrit-Change-Number: 20523 Gerrit-PatchSet: 5 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Sun, 01 Oct 2023 18:32:15 +0000 Gerrit-HasComments: No
