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 10: > Patch Set 10: > > (5 comments) > > Whoops, sorry for not looking at this patch for so long. I was under > impression somebody else is taking care of it. > > Overall looks good to me, just a few nits in the commit message and spacing > in the code. > > Thank you for your contribution! It doesn't matter. I previously thought maybe someone else have some concerns about picking a random replica to scan, so I left this patch here. So glad you think it's ok, I'll fix nits. -- 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: 10 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 06:44:52 +0000 Gerrit-HasComments: No
