easyice opened a new pull request, #13433:
URL: https://github.com/apache/lucene/pull/13433

   If two objects are different, it might have the same hashcode. in this case, 
`Point(-180.0,90.0)` and  `Point(180.0,-90.0)` will have the same hashcode 
value.
   
   ```
   org.apache.lucene.geo.TestPoint > testEqualsAndHashCode FAILED
       java.lang.AssertionError: Values should be different. Actual: -1965031424
           at 
__randomizedtesting.SeedInfo.seed([ADA73B977A63C061:DE69C35A79C6BF50]:0)
           at org.junit.Assert.fail(Assert.java:89)
           at org.junit.Assert.failEquals(Assert.java:187)
           at org.junit.Assert.assertNotEquals(Assert.java:201)
           at org.junit.Assert.assertNotEquals(Assert.java:213)
           at 
org.apache.lucene.geo.TestPoint.testEqualsAndHashCode(TestPoint.java:59)
   ```
   
   ```
   ./gradlew :lucene:core:test --tests 
"org.apache.lucene.geo.TestPoint.testEqualsAndHashCode"  
-Ptests.seed=ADA73B977A63C061
   ```
   
   We had similar issues in the other tests like 
`TestCircle#testEqualsAndHashCode`,  `TestXYPolygon#testEqualsAndHashCode`. it 
seems it also needs to be removed. for random values, the hash code must be 
different is a wrong assumption. 


-- 
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: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to