Fix TestJdbcWithMiniHS2.

Author: ayushtkn

common/src/java/org/apache/hive/common/util/ReflectionUtil.java

@@ -48,7 +48,7 @@ public class ReflectionUtil {
   //       not caching that many constructors.
   // Note that weakKeys causes "==" to be used for key compare; this will only work
   // for classes in the same classloader. Should be ok in this case.
-  private static final Cache<Class<?>, Constructor<?>> CONSTRUCTOR_CACHE =
+  private static Cache<Class<?>, Constructor<?>> CONSTRUCTOR_CACHE =
       CacheBuilder.newBuilder().expireAfterAccess(15, TimeUnit.MINUTES)
                                .concurrencyLevel(64)
                                .weakKeys().weakValues().build();
@@ -174,18 +174,6 @@ public static void setInAllFields(Object object, String field, Object value) {
     }
   }
 
-  public static void setStaticFinalFieldsModifiable(Field field) {
-    try {
-      field.setAccessible(true);
-      VarHandle modifiersHandle = MethodHandles.privateLookupIn(Field.class, MethodHandles.lookup())
-              .findVarHandle(Field.class, "modifiers", int.class);
-      int modifiers = field.getModifiers();
-      modifiersHandle.set(field, modifiers & ~Modifier.FINAL);
-    } catch (NoSuchFieldException | IllegalAccessException e) {
-      throw new RuntimeException("Cannot make static final field %s modifiable".formatted(field));
-    }
-  }
-
   public static List<InstanceField> allDeclaredFieldsOf(Object testInstance) {
     List<InstanceField> result = new ArrayList<>();
     for (Class<?> clazz = testInstance.getClass();

itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniHS2.java

@@ -88,6 +88,8 @@
 import org.junit.Assert;
 import org.junit.BeforeClass;
 import org.junit.Test;
+import sun.misc.Unsafe;
+
 import static org.apache.hadoop.hive.common.repl.ReplConst.SOURCE_OF_REPLICATION;
 
 public class TestJdbcWithMiniHS2 {
@@ -1235,11 +1237,11 @@ private void setReflectionUtilCache() {
     try {
       constructorCacheField = ReflectionUtil.class.getDeclaredField("CONSTRUCTOR_CACHE");
       if (constructorCacheField != null) {
-        ReflectionUtil.setStaticFinalFieldsModifiable(constructorCacheField);
         tmp =
             CacheBuilder.newBuilder().expireAfterAccess(5, TimeUnit.SECONDS).concurrencyLevel(64)
                 .weakKeys().weakValues().build();
-        constructorCacheField.set(tmp.getClass(), tmp);
+        constructorCacheField.setAccessible(true);
+        constructorCacheField.set(null, tmp);
       }
     } catch (Exception e) {
       System.out.println("Error when setting the CONSTRUCTOR_CACHE to expire: " + e);