[SPARK-XXXX][SQL] Address review comments

- Register CrossJoinArrayContainsToInnerJoin in Optimizer.scala batches
- Update benchmark to exclude rule for accurate unoptimized baseline
- Remove Presto references from code comments

Author: yaooqinn

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CrossJoinArrayContainsToInnerJoin.scala

@@ -41,12 +41,10 @@ import org.apache.spark.sql.types._
  * }}}
  *
  * This avoids the O(N*M) cross join by using unnesting and equi-join.
- *
- * Ported from Presto's CrossJoinWithArrayContainsToInnerJoin optimizer rule.
  */
 object CrossJoinArrayContainsToInnerJoin extends Rule[LogicalPlan] with PredicateHelper {
 
-  // Supported element types for the optimization (matching Presto's supported types)
+  // Supported element types for the optimization
   private val supportedTypes: Set[DataType] = Set(
     IntegerType, LongType, StringType, DateType
   )

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala

@@ -261,6 +261,7 @@ abstract class Optimizer(catalogManager: CatalogManager)
     Batch("Optimize One Row Plan", fixedPoint, OptimizeOneRowPlan),
     // The following batch should be executed after batch "Join Reorder" and "LocalRelation".
     Batch("Check Cartesian Products", Once,
+      CrossJoinArrayContainsToInnerJoin,
       CheckCartesianProducts),
     Batch("RewriteSubquery", Once,
       RewritePredicateSubquery,

sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/CrossJoinArrayContainsToInnerJoinBenchmark.scala

@@ -73,8 +73,11 @@ object CrossJoinArrayContainsToInnerJoinBenchmark extends SqlBasedBenchmark {
     items.createOrReplaceTempView("items")
 
     benchmark.addCase("Cross join + array_contains filter (unoptimized)", numIters = 3) { _ =>
-      // Disable the optimization to simulate unoptimized behavior
-      withSQLConf(SQLConf.CROSS_JOINS_ENABLED.key -> "true") {
+      // Disable the optimization to measure the true cross-join+filter baseline
+      withSQLConf(
+        SQLConf.CROSS_JOINS_ENABLED.key -> "true",
+        SQLConf.OPTIMIZER_EXCLUDED_RULES.key ->
+          "org.apache.spark.sql.catalyst.optimizer.CrossJoinArrayContainsToInnerJoin") {
         // This query would be a cross join with filter without optimization
         val df = spark.sql(
           """