Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13287 )
Change subject: KUDU-2711 pt 4. Java support for GetTableLocations optimizations ...................................................................... KUDU-2711 pt 4. Java support for GetTableLocations optimizations This adds support for the optimized GetTableLocations response format added in 586e957f76a547340f2ab93a7eebc3f116ff825e. There's no new tests, but as this changes the way GetTableLocations works, it's tested by all existing tests that do any writes or scans, so it's well-tested by existing tests. Additionally, to test backwards compatibility, I ran the Java client test suite while using a set of binaries compiled without support for the GetTableLocations optimization. Change-Id: I5af146fd1984ce683f056877129506cd2068e0e8 Reviewed-on: http://gerrit.cloudera.org:8080/13287 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Will Berkeley <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java M java/kudu-client/src/main/java/org/apache/kudu/client/GetTableLocationsRequest.java M java/kudu-client/src/main/java/org/apache/kudu/client/LocatedTablet.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/TestAsyncKuduClient.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestRemoteTablet.java 6 files changed, 63 insertions(+), 22 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Will Berkeley: Verified -- To view, visit http://gerrit.cloudera.org:8080/13287 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5af146fd1984ce683f056877129506cd2068e0e8 Gerrit-Change-Number: 13287 Gerrit-PatchSet: 3 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <[email protected]>
