HeartSaVioR opened a new pull request #26709: [SPARK-30075][CORE] Fix the 
hashCode implementation of ArrayKeyIndexType correctly
URL: https://github.com/apache/spark/pull/26709
 
 
   ### What changes were proposed in this pull request?
   
   This patch fixes the bug on ArrayKeyIndexType.hashCode() as it is simply 
calling Array.hashCode which in turn calls Object.hashCode. That should be 
Arrays.hashCode() to reflect the elements in the array.
   
   ### Why are the changes needed?
   
   I've encountered the bug in #25811 and I've split the fix into individual PR 
to speed up reviewing. Without this patch, ArrayKeyIndexType would bring 
various issues when it's used as type of collections.
   
   ### Does this PR introduce any user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   I've skipped adding UT as ArrayKeyIndexType is in test and the patch is 
pretty simple one-liner.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to