Refactor SloppyMath.haversin from SloppyMath to GeoUtils [LUCENE-7164]
Open
legacy-jira-priority:Major
type:enhancement
- Dominant language
- Java
- Stars
- 3.6k
- Forks
- 1.4k
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 88
Description
The `haversin` method is currently in `SloppyMath` but is used specifically for geo. With the new `o.a.l.geo` package in the core module (#8207) it makes more sense to refactor this to `GeoUtils`
---
Migrated from [LUCENE-7164](https://issues.apache.org/jira/browse/LUCENE-7164) by Nick Knize (@nknize)
Contributor guide
Research direction
Start by locating the haversin method in SloppyMath and reviewing GeoUtils in the new o.a.l.geo package. Trace its current callers and related tests before moving the geo-specific method. Done means haversin is provided by GeoUtils, callers still work, and the relevant tests pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- search
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100