aokolnychyi commented on pull request #2255: URL: https://github.com/apache/iceberg/pull/2255#issuecomment-786322311
> I'm trying to think of a better way to structure the ones that take strings. I agree with the idea that it is easier to use that way, but the problem is that we need a way to parse the string into an identifier and to load a catalog... That's the part I am not sure about. I considered using Iceberg's table identifier but is in the `catalog` package, which makes it weird. Plus, one identifier will point to a non-Iceberg table. Apart from that, only implementations will parse identifiers and the syntax could? actually vary from engine to engine. It is unlikely we can make catalog loading and identifier parsing generic. ---------------------------------------------------------------- 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]
