nzw921rx commented on PR #11963: URL: https://github.com/apache/seatunnel/pull/11963#issuecomment-5419584840
@goutamadwant @DanielLeens Thank you for the detailed review🚀 I will make the following adjustments: 1. I agree that the retry mechanism should be removed. It cannot reliably distinguish between a temporary Maven dependency download failure and an actual compilation failure in the PR. 2. Regarding `bypass_actors`, I think we should enable it initially as a fallback in case a bug in Merge Queue prevents PRs from being merged after rollout. Once Merge Queue has proven stable, we can remove the bypass permission. 3. I will also add supplementary documentation. -- 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]
