Tamas Mate has posted comments on this change. ( http://gerrit.cloudera.org:8080/17047 )
Change subject: IMPALA-10161: User LDAP Search bind support ...................................................................... Patch Set 1: Hi Csaba, Thomas, thank you for the reviews, Apologies, I had to re-submit the change under a new change id. Compared to the previous review, this change contains: 1) A factory method that creates the LDAP instance based on the configuration 2) Updated the incorrect flag name in the commit message 3) Cleaned the headers in the webserver.h 4) Ran clang from cli, looks like my IDE was acting up -- To view, visit http://gerrit.cloudera.org:8080/17047 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I978744ad05d9ef408328d1e4dd2d18c329f4d3b7 Gerrit-Change-Number: 17047 Gerrit-PatchSet: 1 Gerrit-Owner: Tamas Mate <tm...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tamas Mate <tm...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Comment-Date: Tue, 09 Feb 2021 12:39:19 +0000 Gerrit-HasComments: No
