goutamadwant commented on PR #11814: URL: https://github.com/apache/seatunnel/pull/11814#issuecomment-5464064563
@DanielLeens addressed the latest review items - replaced the test import with SeaTunnel's `CompletableFuture` - separated initial cursor setup from actual ringbuffer overflow recovery so the first report no longer logs a false warning - restored the scheduler initial delay to `0` to preserve the existing first report timing - added regression coverage for initial cursor setup and later overflow recovery Verified the focused handler tests and the exact import policy test. I also ran the complete `seatunnel-engine-server` package locally with 383 tests and no failures The fixes are pushed in `f83cb2273` and fresh CI is queued -- 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]
