DanielLeens commented on issue #11720:
URL: https://github.com/apache/seatunnel/issues/11720#issuecomment-5234824580

   Thanks for narrowing this down to the shared multi-table sink routing path 
and for connecting it to the older `Integer.MIN_VALUE` fix pattern.
   
   This looks like a real bug in the API layer: `Math.abs(hashCode())` is not 
safe for `Integer.MIN_VALUE`, so with a non-power-of-two 
`multi_table_sink_replica` the routing index can stay negative and fail any 
multi-table sink that hits that key.
   
   I also see two candidate fixes linked here now, #11721 and #11725. To avoid 
splitting review effort, it would be best to converge on one PR path and close 
the duplicate once maintainers choose the cleaner patch and regression-test 
coverage.


-- 
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]

Reply via email to