Alexey Serbin has posted comments on this change.

Change subject: [docs] added info on TSK an CA entries in systable
......................................................................


Patch Set 2:

(3 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 is responsible for removing the data from deleted 
tables
> It's not quite right though; tablet data deletion is done synchronously in 
All right, then I'll just remove this sentence.  Yep, I remember the data is 
deleted from the system table in a synchronous way, and then the 'cleaner' 
updates the metadata in memory.


http://gerrit.cloudera.org:8080/#/c/6111/2/docs/design-docs/master.md
File docs/design-docs/master.md:

PS2, Line 244: contains root
> "contains a root" or "contains the root"
Done


PS2, Line 246: security
> Let's be more specific; this is for authn, right?
This sounds good to me -- will update.

That's for signing server certificates.  The CA cert (without its pairing 
private key) is also used by TLS connection initiators to authenticate the 
server side. For authn tokens we need just private keys, and they are not 
linked to the CA cert or CA private key at all: we just generate private/public 
keypairs out of thin air using OpenSSL crypto library (some system entropy is 
used for creating keys, though).


-- 
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: 2
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: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to