DanielLeens commented on PR #11489:
URL: https://github.com/apache/seatunnel/pull/11489#issuecomment-5332090037

   结论:当前头暂不可交付。
   
   源码正确性:重试用例在任务刚进入 RUNNING 后就直接伪造 checkpoint coordinator 
已就绪,随后第一次请求被拒绝、第二次请求在真实 task-ready 握手尚未完成时启动,最终在 60 秒 checkpoint timeout 后将作业置为 
FAILED。应先等待真实 coordinator ready,再注入一次 false 的前置条件失败,之后恢复 ready 并重试。最小补丁已推送到维护者 
fork:<https://github.com/DanielLeens/seatunnel/commit/fb8618fe928a700cb4611f6618ee0a75e6dd7a44>。
   
   CI 归因:当前头 8f5d15cd 的 unit-test 作业 95686043072 确认失败于 
SavePointBusySourceTest.testSavepointStartPreconditionFailureCanBeRetriedAfterReady;断言等待
 SAVEPOINT_DONE,实际为 FAILED,并非外部基础设施失败。
   
   范围风险:当前头同时加入了 TaskExecutionService 的 cooperative task close 生产逻辑,但该路径不由上述 
savepoint precondition rejection 触发,且没有针对该新行为的回归测试。请将其拆分为独立 
PR,或补充与本问题的因果关系和完整测试证明。
   
   审查与合并门禁:未发现未解决的 review thread,但 reviewDecision 仍为 
CHANGES_REQUESTED,mergeStateStatus 为 BLOCKED。尽管 PR 标示 maintainer 可修改,GitHub 拒绝 
DanielLeens 直接推送到作者分支(403),因此请作者 cherry-pick 上述提交并移除或拆分无关生产改动;新当前头 CI 通过后再复核。


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

Reply via email to