add comments

Author: gh-yzou

build-logic/src/main/kotlin/Utilities.kt

@@ -20,6 +20,11 @@
 import org.gradle.api.Project
 import org.gradle.process.JavaForkOptions
 
+/**
+ * Extract the scala version from polaris spark project, and points the build directory to a sub-dir
+ * that uses scala version as name. The polaris spark project name is in format of
+ * <project>-<sparkVersion>_<scalaVersion>, for example: polaris-spark-3.5_2.12.
+ */
 fun Project.getAndUseScalaVersionForProject(): String {
   val sparkScala = project.name.split("-").last().split("_")
 

plugins/spark/v3.5/integration/src/intTest/java/org/apache/polaris/spark/quarkus/it/SparkCatalogOperationsIT.java

@@ -38,6 +38,11 @@
 
 @QuarkusIntegrationTest
 public class SparkCatalogOperationsIT extends SparkIntegrationBase {
+  /**
+   * This integration directly performs operations using the SparkCatalog instance, instead of going
+   * through Spark SQL interface. Some operations don't have an easy way to trigger through the
+   * SparkSQL, for example, list nested namespaces.
+   */
   private static StructType schema = new StructType().add("id", "long").add("name", "string");
 
   @Test

plugins/spark/v3.5/integration/src/intTest/resources/META-INF/services/org.apache.polaris.service.it.ext.PolarisServerManager

@@ -17,4 +17,4 @@
 # under the License.
 #
 
-org.apache.polaris.service.quarkus.it.QuarkusServerManager
+org.apache.polaris.service.quarkus.it.QuarkusServerManager