Andrew Wong has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/12608 )

Change subject: KUDU-2710: Fix KeepAliveRequest retries
......................................................................

KUDU-2710: Fix KeepAliveRequest retries

Fixes KeepAliveRequest retries by adding a partitionKey
implementation. Without this a null partitionKey is passed
and the client treats this as a master table.

A follow up patch should include fixes to prevent issues
like this in the future and fix any remaining retry issues.
This patch is kept small to ensure easy backports.

Change-Id: I951212ab76079e5788c2870223b45782b16509e7
Reviewed-on: http://gerrit.cloudera.org:8080/12586
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
Reviewed-by: Adar Dembo <[email protected]>
(cherry picked from commit 6302811eb73efdfd2a3da84c25f5d6589302dee1)
Reviewed-on: http://gerrit.cloudera.org:8080/12608
Reviewed-by: Grant Henke <[email protected]>
Tested-by: Andrew Wong <[email protected]>
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java
M java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
3 files changed, 36 insertions(+), 1 deletion(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Grant Henke: Looks good to me, approved
  Andrew Wong: Verified

--
To view, visit http://gerrit.cloudera.org:8080/12608
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.9.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I951212ab76079e5788c2870223b45782b16509e7
Gerrit-Change-Number: 12608
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to