Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8439 )
Change subject: IMPALA-5054: [SECURITY] Enable KRPC w/ TLS in Impala ...................................................................... Patch Set 4: (6 comments) http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/rpc/rpc-mgr.h File be/src/rpc/rpc-mgr.h: http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/rpc/rpc-mgr.h@105 PS3, Line 105: > Seems more appropriate to pass it to the constructor of RpcMgr() instead of Done http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/rpc/rpc-mgr.h@182 PS3, Line 182: > True if TLS is configured for communication between Impalad backend servers Done http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/rpc/rpc-mgr.h@183 PS3, Line 183: /// True if TLS is con > const bool use_tls_; and initialize it in the ctor. Done http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/rpc/rpc-mgr.cc File be/src/rpc/rpc-mgr.cc: http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/rpc/rpc-mgr.cc@88 PS3, Line 88: bld.set_rpc_authentication("required"); : } > This may be worth fixing too in this patch if it's not too complicated. Done http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/rpc/rpc-mgr.cc@117 PS3, Line 117: RETURN_IF_ERROR(service_pool->Init(num_service_threads)); : KUDU_RETURN_IF_ERROR( > One line ? Done http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/util/openssl-util.h File be/src/util/openssl-util.h: http://gerrit.cloudera.org:8080/#/c/8439/3/be/src/util/openssl-util.h@31 PS3, Line 31: // > nit: /// Done -- To view, visit http://gerrit.cloudera.org:8080/8439 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9a14a44fdea9ab668f3714eb69fdb188bce38f5a Gerrit-Change-Number: 8439 Gerrit-PatchSet: 4 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Comment-Date: Tue, 16 Jan 2018 21:38:02 +0000 Gerrit-HasComments: Yes