fishfishfishfishaa commented on PR #8671: URL: https://github.com/apache/paimon/pull/8671#issuecomment-5218869893
Thanks for pointing this out. The main motivation for supporting EndInput in coordinator commit was testability. However, the current implementation directly uses EndInput to avoid final commit loss, which indeed introduces additional complexity in both commit semantics and recovery. I also agree that, for the PIP-30 goal of supporting region failover, batch mode and streaming without checkpointing provide limited practical value. Based on this, I will try to simplify the implementation, narrow the supported scope, and remove the extra EndInput direct-commit path so that the commit and recovery semantics remain clearer. -- 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]
