Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12619 )

Change subject: [master] cache locations assigned to clients
......................................................................


Patch Set 1:

> I wouldn't feel comfortable pushing this into 1.9 without any kind
 > of testing.

Sure, that makes sense.  I'll add a small test targeted for exercising this 
particular piece.  We also have already existing tests for the location 
assignments: both from the Kudu C++ and Java clients.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I12c8952c43a8ad352acd46c8006824b2ad9d1204
Gerrit-Change-Number: 12619
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Wed, 27 Feb 2019 02:26:32 +0000
Gerrit-HasComments: No

Reply via email to