Dan Hecht has posted comments on this change. Change subject: IMPALA-3798: Disable per-split filtering for sequence-based scanners ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3524/1//COMMIT_MSG Commit Message: PS1, Line 19: This bug therefore only shows up when there is >1 scan range per file. > The scan-range-filter path will do most of the work of RangeComplete() for I see. So we won't get stuck forever. But we will still busy-loop while the scan is still in progress (once all scan ranges have been picked up by scanner threads, remaining scanner threads will busy loop which can cause things to slow down drastically). But, okay, I see your point. -- To view, visit http://gerrit.cloudera.org:8080/3524 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4770dd77fd4258c24115d72b572c727b770bd75d Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Henry Robinson <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-HasComments: Yes
