huaxingao commented on pull request #3269:
URL: https://github.com/apache/iceberg/pull/3269#issuecomment-962026695
@rdblue
Yes, I think that should work.
So there is no need to modify `SupportsCatalogOptions`. All I need to do is
to add the new overloaded loadTable methods to pass in time travel info as
follows, right?
```
CatalogV2Util
def loadTable(catalog: CatalogPlugin, ident: Identifier, version: String):
Option[Table]
def loadTable(catalog: CatalogPlugin, ident: Identifier, timestamp: long):
Option[Table]
TableCatalog
default Table loadTable(Identifier ident, String version) {
throw new UnsupportedOperationException("Load table with version is not
supported.");
}
default Table loadTable(Identifier ident, timestamp: long) {
throw new UnsupportedOperationException("Load table with timestamp is
not supported.");
}
```
--
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]
