[
https://issues.apache.org/jira/browse/DRILL-8117?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17603154#comment-17603154
]
ASF GitHub Bot commented on DRILL-8117:
---------------------------------------
kingswanwho commented on code in PR #2499:
URL: https://github.com/apache/drill/pull/2499#discussion_r968518305
##########
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:
@jnturton Hi James, the **_client_** here inherits from BaseTestQuery, which
is DrillClient but not ClientFixture in TestLargeFileCompilation as Paul
suggested. DrillClient doesn't have method like client.alterSystem().
BaseTestQuery is deprecated actually, but not marked as deprecated because it
is still widely used. The best choice here I think is change base test class
from BaseTestQuery to ClusterTest so that we could use client.alterSystem() for
clean and compact code style. There are multiple Inheritance levels here, which
are TestInboundImpersonation -> BaseTestImpersonation -> PlanTestBase ->
BaseTestQuery. Due to PlanTestBase involves many classes. Maybe change
BaseTestImpersonation inheritance from PlanTestBase to ClusterTest directly is
an easier way. However, this PR mainly focus on the deprecated clean up, but
induces more modifications on UT. Is this an appropriate change here?
> Clean up deprecated Apache code in Drill
> ----------------------------------------
>
> Key: DRILL-8117
> URL: https://issues.apache.org/jira/browse/DRILL-8117
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.20.1
> Reporter: Jingchuan Hu
> Priority: Major
> Fix For: 2.0.0
>
>
> Clean up and upgrade deprecated Apache code like:
> Class PathChildrenCache in Class ZookeeperClient and Class StringEscapeUtils
> in Class PlanStringBuilder
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
