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

Elasticsearch 5.x support #6356

Closed
wants to merge 2 commits into from
Closed

Elasticsearch 5.x support #6356

wants to merge 2 commits into from

Conversation

fmauNeko
Copy link

@fmauNeko fmauNeko commented Oct 21, 2016

Hello,

I fixed the queries and the search types which were deprecated since Elasticsearch 2.0.0 and removed with Elasticsearch 5.0.

If you need to play around, here's a test instance : https://metrics.dissidence.ovh/dashboard/db/concorde (Default user/pass). The elasticsearch instance isn't directly open, but you can access it through Kibana at https://kibana.dissidence.ovh/. It is being fed data from Metricbeat through Logstash.

Should fix #5740.

Have a good day !

@torkelo
Copy link
Member

torkelo commented Oct 22, 2016

thanks! this looks pretty great. Think it needs some new unit tests to cover the new scenarios.

@torkelo torkelo self-assigned this Oct 22, 2016
@torkelo torkelo added this to the 4.0.0 milestone Oct 22, 2016
@torkelo torkelo added pr/needs-unit-tests Unit tests are required before merge pr/needs-manual-testing Before merge some help with manual testing & verification is requested labels Oct 22, 2016
@lpic10
Copy link
Contributor

lpic10 commented Oct 24, 2016

Hello @fmauNeko , is there any particular reason to not use the PR #4970?

@fmauNeko
Copy link
Author

Uuuuuh, there's none, actually, I missed it because I was looking at the Datasource: Elasticsearch label and found nothing, sorry.

Also yours is way more complete as I see !

Closing this one.

@fmauNeko fmauNeko closed this Oct 24, 2016
bergquist added a commit that referenced this pull request Dec 9, 2016
@ying-jeanne ying-jeanne added the pr/external This PR is from external contributor label Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/external This PR is from external contributor pr/needs-manual-testing Before merge some help with manual testing & verification is requested pr/needs-unit-tests Unit tests are required before merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: Elasticsearch 5.x support
4 participants