Dan Burkert 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 2: (9 comments) http://gerrit.cloudera.org:8080/#/c/6922/2/docs/prior_release_notes.adoc File docs/prior_release_notes.adoc: Line 40: [[rn_1.3.0]] Todd, do we also need to bring over 1.3.1 release notes, or do those remain only on the 1.3.x branch? 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 'interfaces'. PS2, Line 48: e restricted by a firewall I would also add that it can be safe if authentication and encryption are configured to be required, just so people don't get worried. http://gerrit.cloudera.org:8080/#/c/6922/2/docs/security.adoc File docs/security.adoc: PS2, Line 50: kudu uppercase Kudu PS2, Line 53: , b Same comment about splitting this sentence in two. PS2, Line 56: using '--trust nit: configured using the PS2, Line 58: allows to allow 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 encryption here. PS2, Line 119: kudu Kudu -- 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: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
