Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24111 )
Change subject: IMPALA-14842: Guard doAs and impala.doas.user ...................................................................... IMPALA-14842: Guard doAs and impala.doas.user Prevent setting both doAs and impala.doas.user. They're often set by different actors and we have no use case for setting both. Prevents misuse where override behavior is unexpected. Testing: adds test setting both to LdapHS2Test#testHS2Impersonation Change-Id: If783e48d5782e1c68e2b3fdd9e2aaabb45ddd6ff Reviewed-on: http://gerrit.cloudera.org:8080/24111 Reviewed-by: Fang-Yu Rao <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Joe McDonnell <[email protected]> --- M be/src/service/impala-hs2-server.cc M fe/src/test/java/org/apache/impala/customcluster/LdapHS2Test.java 2 files changed, 27 insertions(+), 2 deletions(-) Approvals: Fang-Yu Rao: Looks good to me, but someone else must approve Impala Public Jenkins: Verified Joe McDonnell: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/24111 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If783e48d5782e1c68e2b3fdd9e2aaabb45ddd6ff Gerrit-Change-Number: 24111 Gerrit-PatchSet: 4 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Xuebin Su <[email protected]>
