Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10510 )
Change subject: IMPALA-5552: Add support for authorized proxy groups ...................................................................... Patch Set 14: (3 comments) http://gerrit.cloudera.org:8080/#/c/10510/13/be/src/service/impala-server.cc File be/src/service/impala-server.cc: http://gerrit.cloudera.org:8080/#/c/10510/13/be/src/service/impala-server.cc@1394 PS13, Line 1394: if (groups.find("*") != groups.end()) return Status::OK(); : : i > nit: single line, no braces (same below) Done http://gerrit.cloudera.org:8080/#/c/10510/13/be/src/service/impala-server.cc@1406 PS13, Line 1406: ); > nit: PrettyPrinter::Print(MonotinicMillis() - start, TUnit::TIME_MS) Done http://gerrit.cloudera.org:8080/#/c/10510/11/tests/authorization/test_authorization.py File tests/authorization/test_authorization.py: http://gerrit.cloudera.org:8080/#/c/10510/11/tests/authorization/test_authorization.py@224 PS11, Line 224: ent_resp = self.hs2 > I see. That coverage could be achieved by modifying the args for the test t That works. Done. -- To view, visit http://gerrit.cloudera.org:8080/10510 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6953f89c293b06b72f523b11802232133d9d6cbb Gerrit-Change-Number: 10510 Gerrit-PatchSet: 14 Gerrit-Owner: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Greg Rahn <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]> Gerrit-Comment-Date: Thu, 07 Jun 2018 01:16:24 +0000 Gerrit-HasComments: Yes
