Andrew Wong has posted comments on this change. Change subject: Kudu Consistency Blog Post Pt1 ......................................................................
Patch Set 12: (17 comments) http://gerrit.cloudera.org:8080/#/c/7019/11/_posts/2017-08-21-kudu-consistency-pt1.md File _posts/2017-08-21-kudu-consistency-pt1.md: PS11, Line 122: nit: extra space 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 nit: consider "that prevent" At first glance, this reads as: "This entails systems that have to deal with (data inconsistencies and corruption) or (prevent them altogether)," instead of, "This entails systems (that have to deal with data inconsistencies and corruption) or (that prevent them altogether)." PS12, Line 43: excelent nit: excellent PS12, Line 43: a nit: capitalize? PS12, Line 54: nit: Dynamo[3]-inspired? Dynamo-inspired[3]? PS12, Line 63: fast analytic workloads nit: consider italicizing this instead of mutable, since immediately after we go into the discussion of what "fast analytical workloads" entail PS12, Line 68: 3 Spanner is [4] PS12, Line 96: trade off nit: trade-off PS12, Line 96: scan time nit: scan-time PS12, Line 97: or scans remove? Ah nevermind. "for scans" perhaps? PS12, Line 117: follow up nit: follow-up PS12, Line 120: mutation nit: mutations PS12, Line 122: user nit: user's PS12, Line 142: its nit: it's PS12, Line 144: to have nit: remove PS12, Line 145: aving nit: remove PS12, Line 146: , nit: remove? -- 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: 12 Gerrit-Project: kudu Gerrit-Branch: gh-pages Gerrit-Owner: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-HasComments: Yes
