StephanEwen commented on pull request #15399:
URL: https://github.com/apache/flink/pull/15399#issuecomment-811835037


   This fix is probably fine. I think it is okay as it was before, because the 
contract with equals is not wrong, just hashCode is less good in spreading 
values over a HashMap.
   
   Not sure if there are any implications for this, because as far as I know 
`equals()` and `hashCode()` are not very well defined here and are mainly used 
for tests.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to