atharvalade opened a new pull request, #3106: URL: https://github.com/apache/iggy/pull/3106
## Which issue does this PR close? Closes #3105 ## Rationale `restart_sink_connector_continues_processing` is flaky in CI due to an insufficient polling budget in `fetch_rows_as`. ## What changed? `fetch_rows_as` used `DEFAULT_POLL_INTERVAL_MS / 5` (10ms) per attempt, giving it a total polling budget of ~1s — far shorter than the ~5s budget of every other polling helper (`wait_for_sink_status`, `wait_for_table`). On loaded CI runners the postgres sink connector cannot always finish processing within 1s after a restart. Changed to use the full `DEFAULT_POLL_INTERVAL_MS` (50ms), matching the other helpers. The function still returns immediately once the expected row count is reached, so passing tests are not slowed down. ## Local Execution - Passed - Pre-commit hooks ran -- 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]
