Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12706 )
Change subject: [known_issues] the scalability of location/rack awareness ...................................................................... Patch Set 3: (8 comments) http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc File docs/known_issues.adoc: http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@143 PS3, Line 143: rack awareness (a.k.a. location awareness) Why bother calling it 'rack awareness' if we call it 'location awareness' everywhere else? http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@144 PS3, Line 144: in Kudu 1.9.0 release Can drop this, I think. http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@145 PS3, Line 145: ConnectToMaster() : requests Can we talk about this in more abstract terms? "If the rate of new clients connecting" perhaps? http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@148 PS3, Line 148: come Nit: arrive http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@150 PS3, Line 150: That might manifest itself in That may manifest as http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@152 PS3, Line 152: alike so http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@157 PS3, Line 157: That allows to keep the This retains the http://gerrit.cloudera.org:8080/#/c/12706/3/docs/known_issues.adoc@161 PS3, Line 161: ** If building from source, introduce the caching of assigned locations: : apply patches `ae6bbcaab` and `4ace91713` from the master branch of the Kudu : git repository, rebuild and deploy kudu-master binaries. This doesn't seem necessary; there are always known issues with patches in newer versions of Kudu or in the master branch. If someone really wants this fixed, they can go looking (or asking). -- To view, visit http://gerrit.cloudera.org:8080/12706 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.9.x Gerrit-MessageType: comment Gerrit-Change-Id: I04dad488a377bf4cd36534d648a69d2fb2444fea Gerrit-Change-Number: 12706 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Greg Solovyev <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Sat, 09 Mar 2019 00:41:47 +0000 Gerrit-HasComments: Yes
