Skip to content

Commit 343053c

Browse files
committed
Fix compilation in Eclipse (backport #59675)
Eclipse was confused by #59583. It can't see a the public inner interface within the superclass. This time. Usually that is fine, but the Eclipse gods don't like this particular code, I guess.
1 parent 76fbe0a commit 343053c

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

server/src/main/java/org/elasticsearch/index/mapper/GeoPointFieldMapper.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,9 @@ public IndexFieldData.Builder fielddataBuilder(String fullyQualifiedIndexName) {
186186

187187
}
188188

189-
protected static class ParsedGeoPoint extends GeoPoint implements ParsedPoint {
189+
// Eclipse requires the AbstractPointGeometryFieldMapper prefix or it can't find ParsedPoint
190+
// See https://bugs.eclipse.org/bugs/show_bug.cgi?id=565255
191+
protected static class ParsedGeoPoint extends GeoPoint implements AbstractPointGeometryFieldMapper.ParsedPoint {
190192
@Override
191193
public void validate(String fieldName) {
192194
if (lat() > 90.0 || lat() < -90.0) {

x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/index/mapper/PointFieldMapper.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,9 @@ public String typeName() {
136136
}
137137
}
138138

139-
protected static class ParsedCartesianPoint extends CartesianPoint implements ParsedPoint {
139+
// Eclipse requires the AbstractPointGeometryFieldMapper prefix or it can't find ParsedPoint
140+
// See https://bugs.eclipse.org/bugs/show_bug.cgi?id=565255
141+
protected static class ParsedCartesianPoint extends CartesianPoint implements AbstractPointGeometryFieldMapper.ParsedPoint {
140142
@Override
141143
public void validate(String fieldName) {
142144
if (Float.isFinite(getX()) == false) {

0 commit comments

Comments
 (0)