[VL] Minor follow-ups for apache#6942

backends-velox/pom.xml

@@ -187,7 +187,7 @@
     <resources>
       <resource>
         <directory>${cpp.releases.dir}</directory>
-        <targetPath>org/apache/gluten/${platform}/${arch}</targetPath>
+        <targetPath>${platform}/${arch}</targetPath>
       </resource>
       <resource>
         <directory>${resource.dir}</directory>

backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxListenerApi.scala

@@ -138,8 +138,8 @@ class VeloxListenerApi extends ListenerApi with Logging {
       JniLibLoader.loadFromPath(libPath, false)
     } else {
       val baseLibName = conf.get(GlutenConfig.GLUTEN_LIB_NAME, "gluten")
-      loader.load(s"$baseLibPackage/${System.mapLibraryName(baseLibName)}", false)
-      loader.load(s"$baseLibPackage/${System.mapLibraryName(VeloxBackend.BACKEND_NAME)}", false)
+      loader.load(s"$platformLibDir/${System.mapLibraryName(baseLibName)}", false)
+      loader.load(s"$platformLibDir/${System.mapLibraryName(VeloxBackend.BACKEND_NAME)}", false)
     }
 
     // Initial native backend with configurations.
@@ -152,18 +152,6 @@ class VeloxListenerApi extends ListenerApi with Logging {
     GlutenRowSplitter.setInstance(new VeloxRowSplitter())
   }
 
-  private lazy val baseLibPackage: String = {
-    val osName = System.getProperty("os.name") match {
-      case n if n.contains("Linux") => "linux"
-      case n if n.contains("Mac") => "darwin"
-      case _ =>
-        // Default to linux
-        "linux"
-    }
-    val arch = System.getProperty("os.arch")
-    s"org/apache/gluten/$osName/$arch"
-  }
-
   private def shutdown(): Unit = {
     // TODO shutdown implementation in velox to release resources
   }
@@ -174,6 +162,17 @@ object VeloxListenerApi {
   //  As spark conf may change when active Spark session is recreated.
   private val driverInitialized: AtomicBoolean = new AtomicBoolean(false)
   private val executorInitialized: AtomicBoolean = new AtomicBoolean(false)
+  private val platformLibDir: String = {
+    val osName = System.getProperty("os.name") match {
+      case n if n.contains("Linux") => "linux"
+      case n if n.contains("Mac") => "darwin"
+      case _ =>
+        // Default to linux
+        "linux"
+    }
+    val arch = System.getProperty("os.arch")
+    s"$osName/$arch"
+  }
 
   private def inLocalMode(conf: SparkConf): Boolean = {
     SparkResourceUtil.isLocalMaster(conf)

backends-velox/src/test/java/org/apache/gluten/test/VeloxBackendTestBase.java

@@ -38,12 +38,12 @@ public static void setup() {
     api.onDriverStart(mockSparkContext(), mockPluginContext());
   }
 
-  private static SparkContext mockSparkContext() {
+  public static SparkContext mockSparkContext() {
     // Not yet implemented.
     return null;
   }
 
-  private static PluginContext mockPluginContext() {
+  public static PluginContext mockPluginContext() {
     return new PluginContext() {
       @Override
       public MetricRegistry metricRegistry() {

backends-velox/src/test/scala/org/apache/gluten/backendsapi/velox/VeloxListenerApiSuite.scala

@@ -0,0 +1,35 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.gluten.backendsapi.velox
+
+import org.apache.gluten.test.VeloxBackendTestBase
+
+import org.scalatest.funsuite.AnyFunSuite
+
+class VeloxListenerApiSuite extends AnyFunSuite {
+  test("Unsupported arch") {
+    System.setProperty("os.arch", "unknown-arch")
+    val api = new VeloxListenerApi()
+    val error = intercept[RuntimeException] {
+      api.onDriverStart(
+        VeloxBackendTestBase.mockSparkContext(),
+        VeloxBackendTestBase.mockPluginContext())
+    }
+    assert(error.getMessage.contains("FileNotFoundException"))
+    assert(error.getMessage.contains("unknown-arch/libgluten.so"))
+  }
+}

package/pom.xml

@@ -98,7 +98,7 @@
       <dependencies>
         <dependency>
           <groupId>org.apache.gluten</groupId>
-	  <artifactId>gluten-delta</artifactId>
+          <artifactId>gluten-delta</artifactId>
           <version>${project.version}</version>
         </dependency>
       </dependencies>
@@ -196,12 +196,12 @@
                   <excludes>
                     <exclude>META-INF/*.SF</exclude>
                     <exclude>META-INF/*.DSA</exclude>
-		    <exclude>META-INF/*.RSA</exclude>
-		    <exclude>META-INF/DEPENDENCIES</exclude>
-		    <exclude>META-INF/LICENSE.txt</exclude>
-		    <exclude>META-INF/NOTICE.txt</exclude>
-		    <exclude>LICENSE.txt</exclude>
-	            <exclude>NOTICE.txt</exclude>
+                    <exclude>META-INF/*.RSA</exclude>
+                    <exclude>META-INF/DEPENDENCIES</exclude>
+                    <exclude>META-INF/LICENSE.txt</exclude>
+                    <exclude>META-INF/NOTICE.txt</exclude>
+                    <exclude>LICENSE.txt</exclude>
+                    <exclude>NOTICE.txt</exclude>
                   </excludes>
                 </filter>
               </filters>