fivetran-kostaszoumpatianos commented on PR #1772:
URL: https://github.com/apache/polaris/pull/1772#issuecomment-2931406463
an alternative at the Polaris level would be to add a:
`boolean realTableExists(...) { ... }` method in the `IcebergCatalog` class.
Then inside `IcebergCatalogHandler`, every time that we would call
`tableExists(...)`, we can call that method (after checking if we can cast the
catalog to an `IcebergCatalog`). That would be in:
- createTableDirectWithWriteDelegation
- stageTableCreateHelper
Also, as far as I understand, the `IcebergCatalogHandler::tableExists`
method that we "expose" in the REST API, uses `loadTable` internally, so this
wouldn't be affected neither by the change above nor by the change proposed in
the current state of this PR.
--
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: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
