From 0129972802d7d97f09a5df100c798bd4411075b6 Mon Sep 17 00:00:00 2001 From: guoqiang Date: Wed, 4 Nov 2026 11:11:38 +0800 Subject: [PATCH] [fix](fe) Fix Ranger Hive unit test initialization on branch-4.1 ### What problem does this PR solve? Related PR: #67414 Problem Summary: The backported Ranger Hive unit test attempts to initialize lifecycleLock by reflection, but RangerHiveAccessController on branch-4.1 does not have that field. This makes testPolicyRequestsUseLowerCaseSelect fail with NoSuchFieldException before exercising row-filter or data-mask requests. Remove the unsupported lock initialization and its unused imports and throws clause while preserving both lowercase select assertions. ### Release note None ### Check List (For Author) - Test: Unit Test - ./run-fe-ut.sh --run org.apache.doris.catalog.authorizer.ranger.hive.RangerHiveAccessControllerTest (2 tests passed); Maven validate (Checkstyle passed). - Behavior changed: No - Does this need documentation: No --- .../ranger/hive/RangerHiveAccessControllerTest.java | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/fe/fe-core/src/test/java/org/apache/doris/catalog/authorizer/ranger/hive/RangerHiveAccessControllerTest.java b/fe/fe-core/src/test/java/org/apache/doris/catalog/authorizer/ranger/hive/RangerHiveAccessControllerTest.java index 95c3cc2a690..844baea3815 100644 --- a/fe/fe-core/src/test/java/org/apache/doris/catalog/authorizer/ranger/hive/RangerHiveAccessControllerTest.java +++ b/fe/fe-core/src/test/java/org/apache/doris/catalog/authorizer/ranger/hive/RangerHiveAccessControllerTest.java @@ -26,9 +26,6 @@ import org.junit.Assert; import org.junit.Test; import org.mockito.Mockito; -import java.lang.reflect.Field; -import java.util.concurrent.locks.ReentrantReadWriteLock; - public class RangerHiveAccessControllerTest { @Test public void testRangerAccessTypeMapping() { @@ -39,12 +36,9 @@ public class RangerHiveAccessControllerTest { } @Test - public void testPolicyRequestsUseLowerCaseSelect() throws Exception { + public void testPolicyRequestsUseLowerCaseSelect() { RangerHiveAccessController controller = Mockito.mock( RangerHiveAccessController.class, Mockito.CALLS_REAL_METHODS); - Field lifecycleLock = RangerHiveAccessController.class.getDeclaredField("lifecycleLock"); - lifecycleLock.setAccessible(true); - lifecycleLock.set(controller, new ReentrantReadWriteLock()); RangerBasePlugin plugin = Mockito.mock(RangerBasePlugin.class); UserIdentity currentUser = UserIdentity.createAnalyzedUserIdentWithIp("user", "%"); RangerAccessRequestImpl rowFilterRequest = new RangerAccessRequestImpl();