Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/3798
Change subject: IMPALA-3857: KuduScanNode race on returning "optional" threads ...................................................................... IMPALA-3857: KuduScanNode race on returning "optional" threads The KuduScanNode could return all active scanner threads when there were no more "optional" tokens available. In this case, any remaining scan ranges wouldn't be picked up and the query would produce incorrect results. This fixes the issue by cleaning up the ScannerThread code and making sure not to exit the last thread. This was tested by running the tpch workload repeatedly under load. That work to incorporate tpch data loading for Kudu is actively being worked on, so this was tested manually. Change-Id: I22adf2109b43b1b37d9a597de85e063431dff155 --- M be/src/exec/kudu-scan-node.cc M be/src/exec/kudu-scan-node.h 2 files changed, 64 insertions(+), 41 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/3798/1 -- To view, visit http://gerrit.cloudera.org:8080/3798 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I22adf2109b43b1b37d9a597de85e063431dff155 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs <[email protected]>
