SEZ9 commented on PR #12366: URL: https://github.com/apache/seatunnel/pull/12366#issuecomment-6458692773
Hi @201811510411lw, thanks for checking Build #27 and reporting back. Your plan is exactly what I would do: wait for #12316 or #12313 to land in `dev`, then do a plain merge of `dev` and rerun `engine-v2-it (11, ubuntu-latest)` together with the cancelled `all-connectors-it-3 (17, ubuntu-latest)` leg. Agreed that no code changes are needed in this PR in the meantime — the `BackpressureSlowSinkIT.testCheckpointsKeepCompletingUnderSustainedBackpressure` failure is outside this diff, and the merge-commit question about `516b855e` vs `ab3fcc09` is already settled by your confirmation that the three PR-owned doc entries were not hand-edited. Remaining asks, just so they are explicit: - Please keep the next sync a plain `dev` merge (no squash/rebase) so the review history on this head stays intact. - Once the rerun completes, drop a short note here with the result of the two legs above, and I will re-check the new head and the Build at that point. Nothing else is open from my side. Thanks for keeping this PR focused. <!-- streview-comment:2080 --> -- 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]
