flyingImer commented on code in PR #3820:
URL: https://github.com/apache/polaris/pull/3820#discussion_r2886897201
##########
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:
yea, lets start a ML thread on this.
IMHO l if this is the pattern Polaris is going forward, we should have the
spec or a rule deterministically suggests the resolving order. And technically,
making them pluggable and consistent ordering across all operations
--
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]
