[GitHub] [iceberg] stevenzwu commented on pull request #8555: Flink: table supplier in writer operator for optional table refresh

["stevenzwu (via GitHub)" <gi...@apache.org>]

stevenzwu commented on PR #8555:
URL: https://github.com/apache/iceberg/pull/8555#issuecomment-1726893429

   > @stevenzwu I reverted things back to use a `Supplier<Table>` instead of a `TableLoader`. The only open comment is whether to remove `isOpen()` from the `TableLoader` interface, currently it is still being used in the new table supplier.
   
   thanks. I think we are very close now. left a few minor comments


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org