gaborkaszab commented on PR #15954: URL: https://github.com/apache/iceberg/pull/15954#issuecomment-4352905222
hey @manuzhang , I think even if the current design doesn't break any engines because of how they use the session catalog, in general we can make this code future proof. Thanks for catching this! I have some other optimizations in mind, but as a pre-requisite I also had to restructure the table cache by not storing the supplier but the 'ingredients' for a table. I created [a refactor PR](https://github.com/apache/iceberg/pull/16173) for that, and it could be useful for this PR as well to simplify the changes. -- 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]
