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 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/8401/2/docs/topics/impala_ssl.xml File docs/topics/impala_ssl.xml: http://gerrit.cloudera.org:8080/#/c/8401/2/docs/topics/impala_ssl.xml@171 PS2, Line 171: p> > This seems redundant, as that's what "Allow any TLS version of 1.2 higher." Done http://gerrit.cloudera.org:8080/#/c/8401/2/docs/topics/impala_ssl.xml@171 PS2, Line 171: p> > Hmm I was trying to come up a subtle way to indicate, "consider using this Done. I'll take out that line. http://gerrit.cloudera.org:8080/#/c/8401/2/docs/topics/impala_ssl.xml@177 PS2, Line 177: lable. The daemons f > The daemon doesn't start. Specifically, we get an error like this: Done http://gerrit.cloudera.org:8080/#/c/8401/2/docs/topics/impala_ssl.xml@188 PS2, Line 188: > nit: whitespace 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: 3 Gerrit-Owner: John Russell <jruss...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Henry Robinson <he...@cloudera.com> Gerrit-Reviewer: John Russell <jruss...@cloudera.com> Gerrit-Reviewer: Michael Brown <mi...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Comment-Date: Mon, 06 Nov 2017 23:53:39 +0000 Gerrit-HasComments: Yes