Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10962 )
Change subject: [java] Add extra info about using multiple clients ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/10962/1/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java File java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java: http://gerrit.cloudera.org:8080/#/c/10962/1/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java@85 PS1, Line 85: that not from this changelist, but just a small nit: that --> when (take it with a grain of salt) http://gerrit.cloudera.org:8080/#/c/10962/1/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java@91 PS1, Line 91: See the options in {@link AsyncKuduClientBuilder} Maybe, it's worth pointing to KuduClientBuilder as well? -- To view, visit http://gerrit.cloudera.org:8080/10962 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I34242862ee6d806b74d7717399e5fd0260dece89 Gerrit-Change-Number: 10962 Gerrit-PatchSet: 1 Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Thu, 19 Jul 2018 00:04:48 +0000 Gerrit-HasComments: Yes