Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18715 )
Change subject: KUDU-2671 fix consistency check in KuduScanner::NextBatch() ...................................................................... KUDU-2671 fix consistency check in KuduScanner::NextBatch() KuduScanner::Data::OpenTablet() can return earlier when the metacache returns a tablet covering the partition key range past the partition key provided. In such case, the 'proxy_' member of KuduScanner::Data isn't set, and that's expected since the upper level code should continue with the next tablet. The new test sub-scenarios introduced in [1] triggered the consistency check on the 'data_->proxy_' member in KuduScanner::NextBatch(), so this patch addresses the issue. Also, this patch re-enables a few test sub-scenarios introduced in [1] and [2]. [1] https://github.com/apache/kudu/commit/b746978c71ce4a95b69d49c43d0ac852909a8b4e [2] https://github.com/apache/kudu/commit/1889d4c44385fec5efeeb2d287d9ab7a3544dcfe Change-Id: Icfbdfac46f35d4d143f37845d158795bf1793da7 Reviewed-on: http://gerrit.cloudera.org:8080/18715 Reviewed-by: Khazar Mammadli <[email protected]> Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> Reviewed-by: Attila Bukor <[email protected]> --- M src/kudu/client/client.cc M src/kudu/client/flex_partitioning_client-test.cc M src/kudu/client/scanner-internal.cc 3 files changed, 17 insertions(+), 24 deletions(-) Approvals: Khazar Mammadli: Looks good to me, but someone else must approve Alexey Serbin: Verified Mahesh Reddy: Looks good to me, but someone else must approve Abhishek Chennaka: Looks good to me, but someone else must approve Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18715 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Icfbdfac46f35d4d143f37845d158795bf1793da7 Gerrit-Change-Number: 18715 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
