simonsssu edited a comment on pull request #1515: URL: https://github.com/apache/iceberg/pull/1515#issuecomment-708200923
@openinx You are right, for batch job, it will break transaction into several small commit because of ProcessingTimer. I also agree to throw exception. when using java api, we can pass stream or batch mode when building FlinkSink, when using in SQL, we may need an API (maybe add in Context ? ) to tell a stream or batch mode. I think it's better to add an API in TableEnvironment so that we can use it when building FlinkSink. ---------------------------------------------------------------- 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]
