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

Software updates/elastic #296

Closed
wants to merge 2 commits into from

Conversation

abhi4578
Copy link
Contributor

  • Correct the geo-shape postman assertions for updated 8.3.3 elasticsearch
  • CI tests might fail, as it points to older version elastic

- Because the newer 'geo_shape' mapping  based on BKD tree
  is more precise (from 50m in older mapping to 1cm precise in newer).
  Refer here for more info: https://discuss.elastic.co/t/issue-after-migration-from-geo-shape-pre-fix-based-to-geo-shape-based-bkd-tree-mapping/311406
@abhi4578
Copy link
Contributor Author

abhi4578 commented Aug 11, 2022

Need to update the elastic client version accordingly:
Low level elastic client to 8.3.3
High level elastic client - Its depreceated for in favor of java-api client, need to use compatible mode @kailash @swaminathanvasanth

@kailash
Copy link
Collaborator

kailash commented Sep 14, 2022

@abhi4578 , we can close this PR since all changes related to this PR are done in #306

@abhi4578
Copy link
Contributor Author

Ok

@abhi4578 abhi4578 closed this Sep 14, 2022
@abhi4578 abhi4578 deleted the software-updates/elastic branch September 14, 2022 10:55
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.

None yet

2 participants