manuzhang opened a new pull request, #16845: URL: https://github.com/apache/iceberg/pull/16845
## Summary - Return delegated session catalog views from `SparkSessionCatalog.listViews` when the wrapped Iceberg catalog does not expose a Spark `ViewCatalog`. - Apply the fix across Spark 3.5, Spark 4.0, and Spark 4.1. - Add versioned regression coverage for the delegated session-catalog `listViews` path. ## Root Cause `SparkSessionCatalog.listViews` called `getSessionCatalog().listViews(namespace)` in the `isViewCatalog()` branch but did not return the delegated result, so it fell through to an empty identifier array. This addresses the independent fix requested in https://github.com/apache/iceberg/pull/16819#discussion_r3425893439. ## Testing - `env GRADLE_USER_HOME=/tmp/gradle ./gradlew -DsparkVersions=3.5 :iceberg-spark:iceberg-spark-3.5_2.12:test --tests org.apache.iceberg.spark.TestSparkSessionCatalog.listViewsReturnsSessionCatalogViews` - `env GRADLE_USER_HOME=/tmp/gradle ./gradlew -DsparkVersions=4.0 :iceberg-spark:iceberg-spark-4.0_2.13:test --tests org.apache.iceberg.spark.TestSparkSessionCatalog.listViewsReturnsSessionCatalogViews` - `env GRADLE_USER_HOME=/tmp/gradle ./gradlew -DsparkVersions=4.1 :iceberg-spark:iceberg-spark-4.1_2.13:test --tests org.apache.iceberg.spark.TestSparkSessionCatalog.listViewsReturnsSessionCatalogViews` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
