Remove reflection based vector API detection

Now that the vector API is a hard requirement, we can avoid probing for
its presence based on reflection.

Author: pettyjamesm

lib/trino-parquet/src/main/java/io/trino/parquet/reader/ColumnReaderFactory.java

@@ -32,6 +32,7 @@
 import io.trino.spi.type.TimestampWithTimeZoneType;
 import io.trino.spi.type.Type;
 import io.trino.spi.type.VarcharType;
+import jdk.incubator.vector.VectorShape;
 import org.apache.parquet.schema.LogicalTypeAnnotation;
 import org.apache.parquet.schema.LogicalTypeAnnotation.DateLogicalTypeAnnotation;
 import org.apache.parquet.schema.LogicalTypeAnnotation.DecimalLogicalTypeAnnotation;
@@ -84,7 +85,7 @@
 
 public final class ColumnReaderFactory
 {
-    private static final int PREFERRED_BIT_WIDTH = getVectorBitSize();
+    private static final int PREFERRED_BIT_WIDTH = VectorShape.preferredShape().vectorBitSize();
 
     private final DateTimeZone timeZone;
     private final boolean vectorizedDecodingEnabled;
@@ -372,16 +373,4 @@ private static boolean isVectorizedDecodingSupported()
         // Graviton 2 machines return false here, whereas x86 and Graviton 3 machines return true
         return PREFERRED_BIT_WIDTH >= 256;
     }
-
-    // get VectorShape bit size via reflection to avoid requiring the preview feature is enabled
-    private static int getVectorBitSize()
-    {
-        try {
-            Class<?> clazz = Class.forName("jdk.incubator.vector.VectorShape");
-            return (int) clazz.getMethod("vectorBitSize").invoke(clazz.getMethod("preferredShape").invoke(null));
-        }
-        catch (Throwable e) {
-            return -1;
-        }
-    }
 }