Abhishek Chennaka has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20923 )
Change subject: KUDU-3526 [java] Scanner should bind with a tserver in java client. ...................................................................... KUDU-3526 [java] Scanner should bind with a tserver in java client. For now, scan requests sent by the java client might fail with scanner not found error. It is because that scanner does not bind with the tserver with which it first communicates. The code in method scanNextRows of java client is still trying to search for the tserver by the locations and selection policy. So if the leader is changed and the next scan request is sent to the new leader, the tserver will respond with the "scanner not found" exception. We should bind the scanner to the tserver, similar to how it is done in the C++ client. Change-Id: I9cf65f4215e99198dd41b43d14e50c8c23b8a9b2 Reviewed-on: http://gerrit.cloudera.org:8080/20715 Tested-by: Kudu Jenkins Reviewed-by: Yifan Zhang <chinazhangyi...@163.com> Reviewed-by: Alexey Serbin <ale...@apache.org> (cherry picked from commit 3895f4f796579071615addfa12d2abf75753b9a1) Reviewed-on: http://gerrit.cloudera.org:8080/20923 Tested-by: Alexey Serbin <ale...@apache.org> Reviewed-by: Abhishek Chennaka <achenn...@cloudera.com> --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java M java/kudu-client/src/main/java/org/apache/kudu/client/RemoteTablet.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduScanner.java 4 files changed, 116 insertions(+), 7 deletions(-) Approvals: Alexey Serbin: Verified Abhishek Chennaka: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20923 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I9cf65f4215e99198dd41b43d14e50c8c23b8a9b2 Gerrit-Change-Number: 20923 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Song Jiacheng <songjiach...@thinkingdata.cn> Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
