SEZ9 commented on PR #12173: URL: https://github.com/apache/seatunnel/pull/12173#issuecomment-5673595091
@Rangsh thanks for the explicit superseded-evidence note. Agreed on all points: - F1–F6 remain closed at `7948e2640`; I re-confirm that push was formatting-only and reopened nothing. - The fork Java 11 Score/Error/CV table from `34354309911` / `34354315647` is treated as superseded on my side. Since both dispatches skipped Checkout/Build PR and only ran `dev @ 75fd4ed`, it is cross-host noise rather than a PR before/after, and I won't reference it as evidence for the growth scenarios. On the same-worker Benchmarks dispatch on `apache/seatunnel` with `pr_number=12173`: that has not been done yet from my side, so the Score/Error/CV follow-up is still pending. Nothing further is needed from you for it — once a dispatch lands, please post the same-worker table here as the replacement for the superseded one. Remaining asks from me: 1. Keep the head at `7948e2640` unchanged unless a re-review requests otherwise, so the pending same-worker numbers line up with what was reviewed. 2. When the same-worker Score/Error/CV is available, post it in this thread so it can be cross-referenced with the closed F1/F3 methodology change. No open code items from my side; the remaining gap is the outstanding `CHANGES_REQUESTED` re-review on the docs change in `9a6077c58`, which is out of my hands. <!-- streview-comment:1050 --> -- 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]
