nzw921rx commented on PR #12173: URL: https://github.com/apache/seatunnel/pull/12173#issuecomment-6208122641
@ > Hi @nzw921rx — friendly follow-up, it's been two weeks since the last activity on this PR, so I wanted to check in rather than let it sit. > > Current state is unchanged and, from a code/review standpoint, complete: > > * Head is still `7948e2640` (no new pushes since 2026-09-13). > * Your 2026-09-10 `CHANGES_REQUESTED` asked not to add the growth scenarios to the refactored benchmark docs. That was addressed in `9a6077c58` (the notes were removed from both `docs/en` and `docs/zh`), and the current head no longer touches `docs/en|zh/engines/zeta/benchmark.md` at all. > * @SEZ9 and @DanielLeens have both confirmed F1–F6 are closed on this head; @DanielLeens has an `APPROVED` on record. > > So the PR is `BLOCKED` purely on that review state, not on any open technical objection. Could you please re-review the current head, or dismiss/clear the request if the docs concern is resolved? > > If you'd prefer I rebase onto the latest `dev` first (the branch is currently 262 commits behind) or split the production `JobHistoryService` change out, just say so and I'll do it — otherwise I'll leave the head untouched so any re-review is against the exact commit that's already been verified. Thanks! Thank you for your work. I will conduct a careful review at an appropriate time. -- 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]
