Skip to content

Fix the version check for LegacyGeoShapeFieldMapper#38547

Merged
nknize merged 3 commits intoelastic:masterfrom
nknize:fix/legacyGeoShapeVersion
Feb 11, 2019
Merged

Fix the version check for LegacyGeoShapeFieldMapper#38547
nknize merged 3 commits intoelastic:masterfrom
nknize:fix/legacyGeoShapeVersion

Conversation

@nknize
Copy link
Copy Markdown
Contributor

@nknize nknize commented Feb 6, 2019

BaseGeoShapeFieldMapper version check needs to be changed from 7.0 to 6.6 to correctly use LegacyGeoShapeFieldMapper for indexes created prior to 6.6.

Version check needs to be changed from 7.0 to 6.6 to correct for backwards compatibility.
@nknize nknize added >bug :Analytics/Geo Indexing, search aggregations of geo points and shapes v7.0.0 labels Feb 6, 2019
@nknize nknize requested a review from jimczi February 6, 2019 23:08
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytics-geo

Copy link
Copy Markdown
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @nknize . I think you can label it as a non-issue since it doesn't affect any released version ?

@nknize
Copy link
Copy Markdown
Contributor Author

nknize commented Feb 11, 2019

@elasticmachine run elasticsearch-ci/packaging-sample

@nknize nknize merged commit d07e61b into elastic:master Feb 11, 2019
nknize added a commit that referenced this pull request Feb 11, 2019
Change version check from 7.0 to 6.6 in BaseGeoShapeFieldMapper to correctly use LegacyGeoShapeFieldMapper for indexes created prior to 6.6.
nknize added a commit that referenced this pull request Feb 11, 2019
Change version check from 7.0 to 6.6 in BaseGeoShapeFieldMapper to correctly use LegacyGeoShapeFieldMapper for indexes created prior to 6.6.
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Feb 12, 2019
* master:
  Enable removal of retention leases (elastic#38751)
  Make the 'get templates' types deprecation message consistent. (elastic#38533)
  Copy retention leases when trim unsafe commits (elastic#37995)
  Fix the version check for LegacyGeoShapeFieldMapper (elastic#38547)
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Feb 12, 2019
* master:
  Enable removal of retention leases (elastic#38751)
  Make the 'get templates' types deprecation message consistent. (elastic#38533)
  Copy retention leases when trim unsafe commits (elastic#37995)
  Fix the version check for LegacyGeoShapeFieldMapper (elastic#38547)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Geo Indexing, search aggregations of geo points and shapes >bug >non-issue v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants