Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/4337
Change subject: IMPALA-4097: Crash in kudu-scan-node-test ...................................................................... IMPALA-4097: Crash in kudu-scan-node-test The kudu-scan-node-test was calling GetNext() with a NULL row batch, which isn't valid. This wasn't failing until a recent code change, and only occasionally due to the timing of scanner threads producing row batches. One batch is empty and the other has 1 row. This test worked when the empty row batch was added to the batch queue first (which usually happened), but the test code couldn't handle the other ordering properly. This fixes the test to be more careful about what is being exercised. Change-Id: I0e30964232ed137fce7a99bf7e9557293b5905c8 --- M be/src/exec/kudu-scan-node-test.cc M be/src/exec/kudu-scan-node.cc 2 files changed, 22 insertions(+), 9 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/37/4337/1 -- To view, visit http://gerrit.cloudera.org:8080/4337 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0e30964232ed137fce7a99bf7e9557293b5905c8 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs <[email protected]>
