rdblue commented on pull request #1783: URL: https://github.com/apache/iceberg/pull/1783#issuecomment-733932509
> if path then handle with iceberg.path option as per above. Seems like should be done as a separate patch? I think that the problem earlier was that once you implement `SupportsCatalogOptions`, you can't also return a table. You have to return a catalog and identifier for everything. I think that will require that all of our catalogs support an identifier that contains a path and use that path to load the table. > Any thoughts on custom catalogs in Spark2.4? We could do this in the `IcebergLoader` for 2.4. I think that could be a nice feature. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
