zhangshenghang commented on PR #12381: URL: https://github.com/apache/seatunnel/pull/12381#issuecomment-5882556938
Thanks @SEZ9 — let's go with your second option: land the ceiling widening as-is and handle F2 and F5 in a follow-up. Both are pre-existing characteristics of the test's choreography rather than anything this diff introduces, and they're orthogonal to the `atMost` ceiling change, so mixing them in here would broaden the PR's scope beyond its one-line purpose. To be explicit for the record: the F2 ordering gap (wall-clock awaits letting rediscovery land after the `second` append) and the F5 diagnostics gap (untilAsserted draining events so the first observation is lost) will be tracked and addressed separately — happy to take that work myself unless someone beats me to it. Thanks for the clear framing of both paths. -- 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]
