FANNG1 commented on PR #11280: URL: https://github.com/apache/gravitino/pull/11280#issuecomment-5065375551
Updated the design based on the earlier feedback. Main changes: - Dropped the `enableRestAccess` flag. A separate plugin (`GravitinoIcebergRestSparkPlugin`) is now the switch — its presence enables the feature, the existing `GravitinoSparkPlugin` behavior is unchanged, and the Iceberg runtime stays isolated. - Discovery is server-authoritative: a new `GET /gravitino/v1/catalogs` on the Iceberg REST server lists the catalogs, so Spark never guesses names — this removes the `static-config-provider` name-mapping problem. - Registration is driven by a name-only `CatalogRegistrationPolicy`, with deterministic precedence: plugin ordering is validated (fail-fast) and user-written `spark.sql.catalog.*` always wins. Could you take another look? @jerryshao @yuqi1129 @roryqi @lasdf1234 @jiangxt2 -- 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]
