Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

support Elasticsearch geo_point and ip data type #719

Merged
merged 1 commit into from
Sep 1, 2020
Merged

support Elasticsearch geo_point and ip data type #719

merged 1 commit into from
Sep 1, 2020

Conversation

penghuo
Copy link
Contributor

@penghuo penghuo commented Aug 27, 2020

Description of changes:

  1. Support Elasticsearch IP, GEO_POINT and HALF_FLOAT data type, but no function support around these data type yet.
  2. Change the formatter from JSONObject to Gson.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@penghuo penghuo requested review from chloe-zh and dai-chen August 27, 2020 23:33
@penghuo penghuo marked this pull request as ready for review August 27, 2020 23:55
@penghuo penghuo self-assigned this Aug 28, 2020
dai-chen
dai-chen approved these changes Aug 28, 2020
View reviewed changes
Copy link
Member

@dai-chen dai-chen 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!

@penghuo penghuo merged commit fe5d539 into opendistro-for-elasticsearch:develop Sep 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dai-chen dai-chen dai-chen approved these changes

@chloe-zh chloe-zh chloe-zh approved these changes

Assignees

@penghuo penghuo

Labels
PPL SQL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@penghuo @chloe-zh @dai-chen