JesusMan0529 opened a new pull request, #12637: URL: https://github.com/apache/seatunnel/pull/12637
### Purpose of this pull request Fixes #12562. Finish the remaining F5 diagnostic issue in `rediscoversFileAfterInactiveCursorClosed`. Poll until events become available, retain them outside the retry callback, and run the count/payload/line assertions once after the wait. Unexpected payload failures now include the actual JSON payload rather than being replaced by a later empty poll. F2 is already addressed by merged #12472, which drives cursor close and rediscovery with a manual clock. This PR preserves that implementation and the existing filesystem wait ceiling, and adds the requested issue pointer beside the ceiling. ### Does this PR introduce _any_ user-facing change? No. Only the existing test changes; production reader behavior is unchanged. ### How was this patch tested? A temporary fault injection changed the expected payload substring to `unexpected-marker` in the existing test, using the real reader/filesystem on Windows with JDK 17: - Before: 10.768-second Awaitility timeout reporting `expected: <1> but was: <0>`; the original payload assertion was lost. - After: 0.784-second assertion failure containing the actual second-line JSON payload. The temporary mutation was removed before the final checks and is not part of this PR. - Root `./mvnw spotless:apply` passed; `git diff --check` passed. - Eight-module reactor `verify` passed: 519 tests, 0 failures/errors, 2 existing Windows skips (517 passed). The Edge Agent connector accounts for 38 tests, including the modified test. - JDK 17 requires `-Dsurefire.jvm.args=--add-opens=java.base/java.net=ALL-UNNAMED` for the existing `ReflectionUtilsTest`; the initial broader run failed on that access restriction, and the reactor passed after adding the test-process flag. - Root `./mvnw -q -DskipTests verify`: still running at publication; no full-root success is claimed. - Docker E2E tests were not run. ### Check list No new dependencies, connector registration changes, or user-facing documentation changes are needed. -- 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]
