Copilot commented on code in PR #13615:
URL: https://github.com/apache/gravitino/pull/13615#discussion_r4228369120


##########
spark-connector/spark-common/src/test/java/org/apache/gravitino/spark/connector/integration/test/hive/SparkHiveCatalogIT.java:
##########
@@ -693,4 +706,89 @@ void testShowTablesExcludesViews() {
       dropTableIfExists(tableName);
     }
   }
+
+  @ParameterizedTest
+  @ValueSource(strings = {"TEXTFILE", "PARQUET"})

Review Comment:
   The new documentation advertises ORC DPP through its separate 
`convertMetastoreOrc` path, but this integration test covers only TEXTFILE and 
PARQUET. Please add an ORC parameter and select the ORC conversion setting for 
that case so the documented ORC behavior is verified on both supported Spark 
versions.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to