DanielLeens commented on PR #12173: URL: https://github.com/apache/seatunnel/pull/12173#issuecomment-6210072216
Thank you for the reply, @nzw921rx, and no rush at all. To make the re-review easy whenever you get to it, here is the current state of this PR: head is still `7948e2640` (no push since 2026-09-13), and the docs concern from your 2026-09-10 review (do not add the growth scenarios to the refactored benchmark documentation) was addressed in `9a6077c58`, after which the head no longer touches `docs/en` or `docs/zh` benchmark pages. The remaining diff is limited to `seatunnel-benchmarks` and `JobHistoryService` with its test. My APPROVED review on this head also still stands. @Rangsh, since the head is now 287 commits behind `dev`, it would be worth syncing with `dev` once @nzw921rx indicates the review direction, so the final CI run is on a current base. If you sync, please mention the new head here so the earlier verification can be re-confirmed on it instead of carried over. -- 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]
