Ambreen Kazi has posted comments on this change.

Change subject: [docs] Add security guide
......................................................................


Patch Set 2:

(17 comments)

http://gerrit.cloudera.org:8080/#/c/6479/2/docs/security.adoc
File docs/security.adoc:

PS2, Line 33: a list of
            : security features which are known deficiencies in the security 
capabilities.
a list of known deficiencies in Kudu's security capabilities.


PS2, Line 53: Secure clusters
            : should
To secure a cluster, always ...


PS2, Line 65: is able to offer
1 word - offers


PS2, Line 77: limited to seven days of validi
rewrite - are only valid for seven days, so that even if a token were 
compromised, it ...


PS2, Line 80: is able to take
1 word - takes


PS2, Line 79: the users of
            : Kudu
just 'users'


PS2, Line 81: needing to communicate
1 word - communicating


PS2, Line 81: ,
remove comma


PS2, Line 110: Secure clusters
             : should 
To secure a cluster, use ..


PS2, Line 117: may 
can


PS2, Line 120: `kudu tserver set_flag`
describe this in words rather than just the command.


PS2, Line 131: based on
using


Line 132: one for each of the two levels. Each access control list specifies a 
comma-separated
.. list 'either' specifies a comma ..


PS2, Line 138: A
lower case


PS2, Line 155: may
can


PS2, Line 158: such as `/metrics`
             : which may be relied upon by monitoring systems to gather metrics 
data.
such as `/metrics`. Monitoring systems rely on these endpoints to gather 
metrics data.


Line 166: will be redacted. This feature can be turned off using the `--redact` 
flag
by default?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iabf60804975dc105243626be48d3a141c9a4dab5
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Ambreen Kazi <ambreen.k...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
Gerrit-HasComments: Yes

Reply via email to