Hao Hao has posted comments on this change.

Change subject: docs: revise security doc based on KUDU-1875 and add release 
notes for 1.4
......................................................................


Patch Set 3:

(8 comments)

http://gerrit.cloudera.org:8080/#/c/6922/2/docs/release_notes.adoc
File docs/release_notes.adoc:

PS2, Line 41: s. But
> I think this will read better if you break the sentence into two after 'int
Done


PS2, Line 48: e restricted by a firewall
> I would also add that it can be safe if authentication and encryption are c
Done


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

PS2, Line 50: Kudu
> uppercase Kudu
Done


PS2, Line 53: . B
> Same comment about splitting this sentence in two.
Done


PS2, Line 56: using the `--t
> nit: configured using the
Done


PS2, Line 58: to all
> to allow
Done


PS2, Line 59: erwise restricted by a firewall,
            : malicious users may be able to gain una
> same thing about clarifying the role of required authentication and encrypt
Done


PS2, Line 119: . Sa
> Kudu
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2e386203f5ed3ef66e2ec136e67738b8c7eb8b1a
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Hao Hao <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to