Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(dsl): Add public data-type GeoPoint and refactor GeoDistanceQuery #277

Merged
merged 2 commits into from
Jul 3, 2023

Conversation

arnoldlacko
Copy link
Collaborator

  • introduced a new data package to hold types to be used as field-types in case classes representing documents
  • introduced GeoPoint as a public data-type under a new data package
  • introduced GeoHash within the query package used for querying by geo-hash
  • refactored GeoDistanceQuery to rely on these types
  • made the distance parameter on GeoDistanceQuery non-optional

@arnoldlacko arnoldlacko changed the title (dsl): added public type GeoPoint and refactored GeoDistanceQuery (dsl): added public data-type GeoPoint and refactored GeoDistanceQuery Jul 2, 2023
@arnoldlacko arnoldlacko force-pushed the expose-geo-point-as-a-public-datatype branch from 8567562 to 1dc7dd5 Compare July 2, 2023 15:29
@arnoldlacko arnoldlacko force-pushed the expose-geo-point-as-a-public-datatype branch from 1dc7dd5 to 30f1fc2 Compare July 2, 2023 15:30
@drmarjanovic drmarjanovic changed the title (dsl): added public data-type GeoPoint and refactored GeoDistanceQuery (dsl): Add public data-type GeoPoint and refactored GeoDistanceQuery Jul 3, 2023
@drmarjanovic drmarjanovic changed the title (dsl): Add public data-type GeoPoint and refactored GeoDistanceQuery (dsl): Add public data-type GeoPoint and refactor GeoDistanceQuery Jul 3, 2023
@drmarjanovic drmarjanovic merged commit b567c7d into main Jul 3, 2023
13 checks passed
@drmarjanovic drmarjanovic deleted the expose-geo-point-as-a-public-datatype branch July 3, 2023 09:07
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.

2 participants