[PR] [SEDONA-690] Set default metric to use Haversine for KNN join and code refactoring [sedona]

["zhangfengcdt (via GitHub)" <gi...@apache.org>]

zhangfengcdt opened a new pull request, #1909:
URL: https://github.com/apache/sedona/pull/1909

   
   ## Did you read the Contributor Guide?
   
   - Yes, I have read the [Contributor Rules](https://sedona.apache.org/latest/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/latest/community/develop/)
   
   ## Is this PR related to a ticket?
   
   - Yes, and the PR name follows the format `[SEDONA-XXX] my subject`.
   
   ## What changes were proposed in this PR?
   
   This PR change the default distance metric to use Haversine distance when use_spheroid = true. It also refactors the join index judgement code to be more readable.
   
   ## How was this patch tested?
   KNNTestSuit
   
   ## Did this PR include necessary documentation updates?
   
   - No, this PR does not affect any public API so no need to change the documentation.
   


-- 
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-unsubscribe@sedona.apache.org

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

Re: [PR] [SEDONA-690] Set default metric to use Haversine for KNN join and code refactoring [sedona]

["jiayuasu (via GitHub)" <gi...@apache.org>]

jiayuasu merged PR #1909:
URL: https://github.com/apache/sedona/pull/1909


-- 
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-unsubscribe@sedona.apache.org

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