Alexey Serbin has posted comments on this change. Change subject: [docs] added info on TSK an CA entries in systable ......................................................................
Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6111/1/docs/design-docs/master.md File docs/design-docs/master.md: Line 107: A "Cleaner" process will be responsible for removing the data from deleted tables > Pretty sure we're handling this correctly now? ok, I'll remove this TODO and turn the sentence into the present tense. PS1, Line 244: root CA certificate > Isn't that redundant? root Certificate Authority certificate I'm not sure I understand which redundant part you are pointing at. 'Certificate Authority certificate' means the certificate which is used to verify that some other certificate (like server cert) is signed with corresponding private key. 'Root' means we use our own cert chain and our CA is not a subordinate one. 'Certificate Authority' is just an entity authorized to issue X509 certificates. We authorized some component of leader master server to act as CA :) I think 'root CA certificate' is not redundant. Let me know if you feel strong about this -- probably we can re-phrase it somehow. PS1, Line 249: yet > Seems redundant. Done PS1, Line 250: the > remove Done -- To view, visit http://gerrit.cloudera.org:8080/6111 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3c75d792981c3152f5844a2ea6934c0ce0032522 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
