David Ribeiro Alves has posted comments on this change.

Change subject: Kudu Consistency Blog Post Pt1
......................................................................


Patch Set 11:

(23 comments)

http://gerrit.cloudera.org:8080/#/c/7019/12/_posts/2017-08-21-kudu-consistency-pt1.md
File _posts/2017-08-21-kudu-consistency-pt1.md:

PS12, Line 35:  prevent
> +1
removed this


PS12, Line 43: a
> nit: capitalize?
this is his "handle" which is not capitalized. I switched this to his actual 
name


PS12, Line 43: excelent
> nit: excellent
Done


PS12, Line 54:  
> another option might be: ... systems inspired by _Dynamo_[3], ...
Done


PS12, Line 54:  
> nit: Dynamo[3]-inspired? Dynamo-inspired[3]?
Done


PS12, Line 57: availability
> This is a touch confusing because availability is being maximized by minimi
well the point is that not having insertion errors and/or having constant low 
latencies _is_ having high availability. added "related to" to make this 
clearer.


PS12, Line 60: ,
> Remove
Done


PS12, Line 63: fast analytic workloads
> nit: consider italicizing this instead of mutable, since immediately after 
I italicized the whole thing


PS12, Line 68: 3
> Spanner is [4]
Done


PS12, Line 96: scan time
> nit: scan-time
Done


PS12, Line 96: trade off
> nit: trade-off
Done


PS12, Line 97:  or scans
> remove?
Done


PS12, Line 104:  so
> Consider starting a new sentence from here for better readability:
Done


PS12, Line 117: follow up
> nit: follow-up
Done


PS12, Line 119: wall time
> wall clock time ?
clock time might be confusing here since in future posts we'll be introducing 
multiple clocks. wall time is well established enough to make sense IMO


PS12, Line 120: mutation
> nit: mutations
Done


PS12, Line 122: user
> nit: user's
Done


PS12, Line 142: its
> nit: it's
Done


PS12, Line 144: to have
> nit: remove
Done


PS12, Line 145: aving 
> nit: remove
Done


PS12, Line 146: ,
> nit: remove?
Done


PS12, Line 152: programatically
> typo: programmatically
Done


Line 171: [[3]](www.allthingsdistributed.com/files/amazon-dynamo-sosp2007.pdf): 
Giuseppe DeCandia, Deniz Hastorun, Madan Jampani, Gunavardhan Kakulapati, 
Avinash Lakshman, Alex Pilchin, Swaminathan Sivasubramanian, Peter Vosshall, 
and Werner Vogels. 2007. Dynamo: amazon's highly available key-value store. In 
Proceedings of twenty-first ACM SIGOPS symposium on Operating systems 
principles (SOSP '07). ACM, New York, NY, USA
> nit: consider adding the protocol prefix 'http://'
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icaec0c8ace20651a65901a8e1786e785265540d1
Gerrit-PatchSet: 11
Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-Owner: David Ribeiro Alves <davidral...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com>
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
Gerrit-HasComments: Yes

Reply via email to