Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18222 )
Change subject: [java] KUDU-3349 Fix the failure to demote a leader ...................................................................... [java] KUDU-3349 Fix the failure to demote a leader KuduScanToken gets a wrong tserver's uuid whose format is something like: '<ByteString@6dffd497 size=32 contents="fc07f681d3ea4bab9bc5ec8090ab9437">', the expected uuid should be "fc07f681d3ea4bab9bc5ec8090ab9437". This issue caused RemoteTablet to fail to demote a leader, and the java client always sends write ops to the demoted leader. As a result, there are a lot of "PendingErrors overflowed. Failed to write at least 1000 rows to Kudu". After this fix, the write ops, especially the deleting, will be faster. Change-Id: I2974b6ec2cec2f0120b113d1bcf89fe3793a1ec5 Reviewed-on: http://gerrit.cloudera.org:8080/18166 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit 90895ce76590f10730ad7aac3613b69d89ff5422) Reviewed-on: http://gerrit.cloudera.org:8080/18222 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Attila Bukor <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/KuduScanToken.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestScanToken.java 2 files changed, 172 insertions(+), 24 deletions(-) Approvals: Alexey Serbin: Verified Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18222 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.16.x Gerrit-MessageType: merged Gerrit-Change-Id: I2974b6ec2cec2f0120b113d1bcf89fe3793a1ec5 Gerrit-Change-Number: 18222 Gerrit-PatchSet: 2 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: Hongjiang Zhang <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
