Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12158 )
Change subject: KUDU-2348: Pick a random replica in RemoteTablet.java ...................................................................... Patch Set 11: (5 comments) http://gerrit.cloudera.org:8080/#/c/12158/10//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/12158/10//COMMIT_MSG@15 PS10, Line 15: fix > nit: fixes Done http://gerrit.cloudera.org:8080/#/c/12158/10//COMMIT_MSG@16 PS10, Line 16: initialize > nit: initializer Done http://gerrit.cloudera.org:8080/#/c/12158/10/java/kudu-client/src/main/java/org/apache/kudu/client/RemoteTablet.java File java/kudu-client/src/main/java/org/apache/kudu/client/RemoteTablet.java: http://gerrit.cloudera.org:8080/#/c/12158/10/java/kudu-client/src/main/java/org/apache/kudu/client/RemoteTablet.java@201 PS10, Line 201: a > nit: missing spaces around '%' Done http://gerrit.cloudera.org:8080/#/c/12158/10/java/kudu-client/src/main/java/org/apache/kudu/client/RemoteTablet.java@216 PS10, Line 216: ( > nit: missing spaces around '%' Done http://gerrit.cloudera.org:8080/#/c/12158/10/java/kudu-client/src/main/java/org/apache/kudu/client/RemoteTablet.java@220 PS10, Line 220: n > nit: missing spaces around '%' Done -- To view, visit http://gerrit.cloudera.org:8080/12158 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3d70e45d4c9532bb32223c1dddd0936b4ff8fd99 Gerrit-Change-Number: 12158 Gerrit-PatchSet: 11 Gerrit-Owner: Yifan Zhang <chinazhangyi...@163.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com> Gerrit-Comment-Date: Sat, 24 Aug 2019 07:08:10 +0000 Gerrit-HasComments: Yes