Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12119 )

Change subject: [blog] a blogpost about location awareness in Kudu
......................................................................


Patch Set 10:

(8 comments)

Just a few small comments. I think this is almost good to go.

http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md
File _posts/2019-04-16-location-awareness.md:

http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@48
PS10, Line 48: leverage the hierarchical structure
             : in future releases.
Is there a Jira tracking this?


http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@62
PS10, Line 62: (the latter renders the tablet
             : server unusable to Kudu clients).
How should users detect this? Does the tablet server fail? Is there a log to 
look for?


http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@73
PS10, Line 73: assigned client's
client's assigned


http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@81
PS10, Line 81: follow the following
follow the following just sounds funny because it's two forms for "follow". How 
about "adhere to the following"?


http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@85
PS10, Line 85: as _replica placement policy_
as the replica placement policy


http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@87
PS10, Line 87: the
remove "the"


http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@103
PS10, Line 103: in future releases that might be done automatically in 
background
Is thee a jira to track this?


http://gerrit.cloudera.org:8080/#/c/12119/10/_posts/2019-04-16-location-awareness.md@368
PS10, Line 368: References
Now that these are linked inline, are they needed here too?



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

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: I10b30a80d5661fb889a11285b8118cdea1a87cd2
Gerrit-Change-Number: 12119
Gerrit-PatchSet: 10
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gsolov...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
Gerrit-Comment-Date: Wed, 17 Apr 2019 18:36:55 +0000
Gerrit-HasComments: Yes

Reply via email to