iting0321 commented on code in PR #3820:
URL: https://github.com/apache/polaris/pull/3820#discussion_r2877353899
##########
plugins/spark/v3.5/spark/src/main/java/org/apache/polaris/spark/SparkCatalog.java:
##########
@@ -143,7 +143,29 @@ public Table loadTable(Identifier ident) throws
NoSuchTableException {
try {
return this.icebergsSparkCatalog.loadTable(ident);
} catch (NoSuchTableException e) {
- return this.polarisSparkCatalog.loadTable(ident);
+ // For generic tables, first try to load from Polaris to check the format
Review Comment:
The design resolves the provider via
`polarisSparkCatalog.loadGenericTable(ident)` first, then delegates based on
the result, no speculative multi-catalog probing. Delegation is deterministic,
it only happens once the provider is known, and keeping Polaris as the single
entry point.
Happy to discuss the broader routing strategy on the dev mailing list if
needed.
--
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]
