DanielLeens commented on PR #11721: URL: https://github.com/apache/seatunnel/pull/11721#issuecomment-5612006176
@SEZ9 Agreed on both — nothing further needed from me either. My 08-24 approval stands against the current head; no blockers on my side. +1 on your non-blocking ask: if @SEPURI-SAI-KRISHNA pushes again, a short comment at the inline `(hash & Integer.MAX_VALUE) % n` expression pointing at the `HashUtils.bucketIndex` follow-up issue would be a nice touch, but I wouldn't hold the PR open for it either. -- 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]
