SEPURI-SAI-KRISHNA commented on PR #12049: URL: https://github.com/apache/seatunnel/pull/12049#issuecomment-5569796172
Thanks for the thorough re-review, and for re-deriving both tests against the enumerator's actual control flow rather than reading them off the commit message. One note in case it is useful to anyone reading later: the earlier e2e failure on `9f1c51ce9` could not have involved this PR's change even in principle. `AbstractAzureCosmosDBIT.readRows`, the frame in that stack, constructs `AzureCosmosDBSourceReader` directly and builds its own `new AzureCosmosDBSourceSplit(0)`, so it never instantiates the enumerator at all. Combined with the current head's failure happening at class-setup time before any test method runs, both failures sit entirely outside the changed code path. Rerunning the failed job as you suggested. If the Cosmos emulator keeps timing out on unrelated PRs, its startup budget in CI probably deserves its own issue rather than being folded into this one. -- 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]
