honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-02-17 10:25:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix incorrect geohash for lat 90, lon 180 (#29256)

Browse Source 
Due to special treatment for the 0xFFFFFF... value in GeoHashUtils'
encodeLatLon method, the hashcode for lat 90, lon 180 is incorrectly
encoded as `"000000000000"` instead of "zzzzzzzzzzzz". This commit
removes the special treatment and fixes the issue.

Closes #22163
This commit is contained in:
Igor Motov 2018-03-29 09:23:43 -04:00 committed by GitHub
parent b6568d0cfd
commit 04d0edc8ee
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 14 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
server/src/main/java/org/elasticsearch/common/geo/GeoHashUtils.java
View File

@ -57,11 +57,7 @@ public class GeoHashUtils {
* 31 bit encoding utils * * 31 bit encoding utils *
*************************/ *************************/
public static long encodeLatLon(final double lat, final double lon) { public static long encodeLatLon(final double lat, final double lon) {
long result = MortonEncoder.encode(lat, lon); return MortonEncoder.encode(lat, lon) >>> 2;
if (result == 0xFFFFFFFFFFFFFFFFL) {
return result & 0xC000000000000000L;
}
return result >>> 2;
} }
/** /**

14
server/src/test/java/org/elasticsearch/common/geo/GeoHashTests.java
View File

@ -25,7 +25,7 @@ import org.elasticsearch.test.ESTestCase;
* Tests for {@link org.elasticsearch.common.geo.GeoHashUtils} * Tests for {@link org.elasticsearch.common.geo.GeoHashUtils}
*/ */
public class GeoHashTests extends ESTestCase { public class GeoHashTests extends ESTestCase {
public void testGeohashAsLongRoutines() { public void testGeohashAsLongRoutines() {
final GeoPoint expected = new GeoPoint(); final GeoPoint expected = new GeoPoint();
final GeoPoint actual = new GeoPoint(); final GeoPoint actual = new GeoPoint();
//Ensure that for all points at all supported levels of precision //Ensure that for all points at all supported levels of precision
@ -70,4 +70,16 @@ public class GeoHashTests extends ESTestCase {
assertEquals(expectedLatDiff, bbox.maxLat - bbox.minLat, 0.00001); assertEquals(expectedLatDiff, bbox.maxLat - bbox.minLat, 0.00001);
assertEquals(hash, GeoHashUtils.stringEncode(bbox.minLon, bbox.minLat, level)); assertEquals(hash, GeoHashUtils.stringEncode(bbox.minLon, bbox.minLat, level));
} }
public void testGeohashExtremes() {
assertEquals("000000000000", GeoHashUtils.stringEncode(-180, -90));
assertEquals("800000000000", GeoHashUtils.stringEncode(-180, 0));
assertEquals("bpbpbpbpbpbp", GeoHashUtils.stringEncode(-180, 90));
assertEquals("h00000000000", GeoHashUtils.stringEncode(0, -90));
assertEquals("s00000000000", GeoHashUtils.stringEncode(0, 0));
assertEquals("upbpbpbpbpbp", GeoHashUtils.stringEncode(0, 90));
assertEquals("pbpbpbpbpbpb", GeoHashUtils.stringEncode(180, -90));
assertEquals("xbpbpbpbpbpb", GeoHashUtils.stringEncode(180, 0));
assertEquals("zzzzzzzzzzzz", GeoHashUtils.stringEncode(180, 90));
}
} }