hililiwei commented on PR #6614:
URL: https://github.com/apache/iceberg/pull/6614#issuecomment-1399718452

   > I think the problem is that the `TableLoader` is taking ownership of the 
catalog and closing it. That seems incorrect to me.
   
   So from that point of view, cloning tables doesn't seem like a good idea, 
they cannot share resources. . We should keep the catalog available in the 
loader, or create a statically shared catalog.
   Alternatively, we could try to get rid of tableLoader altogether and switch 
to catalog, but this would be a compatibility breaking change.
   


-- 
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]

Reply via email to