John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8401 )
Change subject: IMPALA-5473: [DOCS] Document TLS min version & cipher options ...................................................................... Patch Set 2: (4 comments) I included the "long-form" description of the RHEL/CentOS issue from Sailesh's email. http://gerrit.cloudera.org:8080/#/c/8401/1/docs/topics/impala_ssl.xml File docs/topics/impala_ssl.xml: http://gerrit.cloudera.org:8080/#/c/8401/1/docs/topics/impala_ssl.xml@145 PS1, Line 145: . In <keyword keyref="impala210_full"/>, you can use startup : options for the <cmdname>impal > I assume you meant catalogd and statestored in 2 of these? Done http://gerrit.cloudera.org:8080/#/c/8401/1/docs/topics/impala_ssl.xml@158 PS1, Line 158: <p> > mention this the default may be? Right, the default is stated on the next line. http://gerrit.cloudera.org:8080/#/c/8401/1/docs/topics/impala_ssl.xml@177 PS1, Line 177: s of <keyword keyref="impala210_full"/>, TLSv1.2 may not work for > Elaborate on the default state a little bit: "Default is empty and then Imp Done http://gerrit.cloudera.org:8080/#/c/8401/1/docs/topics/impala_ssl.xml@182 PS1, Line 182: </p> > Could you also please add a note saying that TLSv1.2 may not work on CentOS Done -- To view, visit http://gerrit.cloudera.org:8080/8401 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia1705262f8c01e38c616541d1c48f5d0cad5498e Gerrit-Change-Number: 8401 Gerrit-PatchSet: 2 Gerrit-Owner: John Russell <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: John Russell <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Fri, 27 Oct 2017 19:35:55 +0000 Gerrit-HasComments: Yes
