abdessalems commented on PR #11727: URL: https://github.com/apache/seatunnel/pull/11727#issuecomment-5462628507
Good catch on the iteration count, @DanielLeens — that wasn't intentional. 410b47409 is a merge of dev into the branch and ITERATIONS got dropped from 30 to 5 in the conflict resolution rather than as a deliberate change. I'll restore it to 30; the two tests run in about 13s at that count locally, so there's no runtime reason to lower it. Thanks for the thorough passes on this one throughout — and agreed on syncing with @waterWang so the identity-checked removal lands once rather than twice. -- 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]
