kingswanwho commented on code in PR #2499:
URL: https://github.com/apache/drill/pull/2499#discussion_r851795975
##########
exec/java-exec/src/test/java/org/apache/drill/exec/impersonation/TestInboundImpersonation.java:
##########
@@ -156,22 +159,25 @@ public void unauthorizedTarget() throws Exception {
@Test
public void invalidPolicy() throws Exception {
- thrownException.expect(new
UserExceptionMatcher(UserBitShared.DrillPBError.ErrorType.VALIDATION,
- "Invalid impersonation policies."));
+ String query = "ALTER SYSTEM SET `%s`='%s'";
Review Comment:
@paul-rogers Hi Paul, I am sorry that I dragged this PR out too long. I
replaced hard-code to FixtureClient, but UT failed due to the Error:
`java.lang.AssertionError: unexpected exception type thrown;
expected:<org.apache.drill.common.exceptions.UserException> but
was:<java.lang.IllegalStateException>
at
org.apache.drill.exec.impersonation.TestInboundImpersonation.invalidProxy(TestInboundImpersonation.java:186)
Caused by: java.lang.IllegalStateException:
org.apache.drill.common.exceptions.UserRemoteException: PERMISSION ERROR:
Cannot change option exec.impersonation.inbound_policies in scope SESSION`
I think the two clients cannot exist at the same time, so I changed back to
the hard-code way.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
