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 support for alterting #5948

Merged
merged 1 commit into from
Sep 21, 2016
Merged

elasticsearch support for alterting #5948

merged 1 commit into from
Sep 21, 2016

Conversation

DEvil0000
Copy link

Issue #5858 #5893
This is the first go code I was ever writing. It is very prototypical and may suffer form bad practice patterns.
Most elasticsearch queries should work.

Not (fully) working:

  • Filters buckets
  • GeoHashGrid buckets
  • ExtendedStats metric (result parsing)
  • Percentiles metric (result parsing)
  • Raw Document
  • Queries without DateHistogram as last bucket

@CLAassistant
Copy link

CLAassistant commented Sep 1, 2016

CLA assistant check
All committers have signed the CLA.

@torkelo torkelo added the area/alerting Grafana Alerting label Sep 5, 2016
@bergquist bergquist merged commit 53522f6 into grafana:alerting-elasticsearch Sep 21, 2016
@bergquist
Copy link
Contributor

Merging this as intial work on alerting for elasticsearch.

@harshvladha
Copy link

@bergquist can I use this merge request with current master branch, while building grafana from source? I don't need above query supports as of now, will it have any issues after building with this branch?

@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
area/alerting Grafana Alerting pr/external This PR is from external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants