Skip to content

Conversation

@russcam
Copy link
Contributor

@russcam russcam commented Apr 26, 2018

This commit deprecates IgnoreUnmapped on IGeoShape by applying Obsolete attribute.
IgnoreUnmapped exists at the same level as _name and boost on a geo_shape query.

Map IgnoreUnmapped from IGeoShape.IgnoreUnmapped to IGeoShapeQuery.IgnoreUnmapped if
assigned.

This commit deprecates IgnoreUnmapped on IGeoShape by applying Obsolete attribute.
IgnoreUnmapped exists at the same level as _name and boost on a geo_shape query.

Map IgnoreUnmapped from IGeoShape.IgnoreUnmapped to IGeoShapeQuery.IgnoreUnmapped if
assigned.
@russcam
Copy link
Contributor Author

russcam commented Apr 26, 2018

When ported to master, Obsolete attributes can be removed as part of refactoring in #3213

@russcam russcam requested a review from codebrain April 26, 2018 05:02
Copy link
Contributor

@codebrain codebrain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - nice move!

russcam added a commit that referenced this pull request Apr 26, 2018
* Support serialization of geo shapes on documents

Fixes #3100
Fixes #3096

* GeometryCollection implements IGeoShape

This commit changes GeometryCollection to implement IGeoShape, to allow a document to have an
IGeoShape member and accept a GeometryCollection. IGeoShape.Type is implemented explicitly so
that the existing Type member continues to be the implicit implementation of IGeometryCollection.

The ideal scenario would have been for IGeometryCollection to implement IGeoShape however this
would break backwards binary compatibility, since both interfaces have a Type member.

(cherry picked from commit 96c0d4d)

This commit includes changes from PR #3214, which were required to address incorrect
serialization introduced in 6.x
@russcam russcam merged commit 1055eb2 into 6.x Apr 26, 2018
@russcam russcam deleted the bug/geoshape-ignoreunmapped branch April 28, 2018 11:34
russcam added a commit that referenced this pull request May 1, 2018
* Obsolete IgnoreUnmapped on IGeoShape

This commit removes IgnoreUnmapped on IGeoShape.
IgnoreUnmapped exists at the same level as _name and boost on a geo_shape query.

(cherry picked from commit 1055eb2)
russcam added a commit that referenced this pull request May 2, 2018
* Obsolete IgnoreUnmapped on IGeoShape

This commit removes IgnoreUnmapped on IGeoShape.
IgnoreUnmapped exists at the same level as _name and boost on a geo_shape query.

(cherry picked from commit 1055eb2)
russcam added a commit that referenced this pull request May 2, 2018
* Obsolete IgnoreUnmapped on IGeoShape

This commit removes IgnoreUnmapped on IGeoShape.
IgnoreUnmapped exists at the same level as _name and boost on a geo_shape query.

(cherry picked from commit 1055eb2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants