SEPURI-SAI-KRISHNA commented on PR #12049: URL: https://github.com/apache/seatunnel/pull/12049#issuecomment-5572155096
This one is ready for a committer whenever there is time. `Build` is green on `635d7874`. The earlier Cosmos DB emulator timeouts were infrastructure rather than code: `AbstractAzureCosmosDBIT.readRows` constructs the reader and its split directly and never instantiates the enumerator, so the changed method is not on that path at all. The rerun passed. The production change is one import and one line in `getSplitOwner`, replacing the local `Math.abs` based modulo with `HashUtils.bucketIndex`, the same migration as #11937 and #11987. @DanielLeens reviewed it and approved, and the two regression tests he suggested are included. @davidzollo would you mind taking a look? -- 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]
