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

Cancel underlying aggregation query if aggregation is modified #17846

Closed
alexfrancoeur opened this issue Apr 23, 2018 · 1 comment · Fixed by #42035
Closed

Cancel underlying aggregation query if aggregation is modified #17846

alexfrancoeur opened this issue Apr 23, 2018 · 1 comment · Fixed by #42035
Labels
Feature:Search Querying infrastructure in Kibana Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:enhancement

Comments

@alexfrancoeur
Copy link

It is possible that an aggregation defined in visualize can have more impact on Elasticsearch than you intended. If an end user decides to modify an aggregation when defining a visualization, we should automatically cancel the previous aggregation query.

Utilizing the new ability to associate an ID with tasks (elastic/elasticsearch#27764) and the task management API, we should be able to automatically cancel a running query if the user has redefined the aggregation for a visualization.

Related to #14544, #17265, #17844, #17845

@alexfrancoeur alexfrancoeur changed the title Cancel underlying aggregation if aggregation is modified Cancel underlying aggregation query if aggregation is modified Apr 23, 2018
@timroes timroes added Feature:Search Querying infrastructure in Kibana Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:enhancement labels Apr 25, 2018
@alexfrancoeur
Copy link
Author

Related to:
[Management] List / Terminate Running Tasks #18829
add search ids to requests made to elasticsearch #16493

@timroes timroes added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Sep 16, 2018
@timroes timroes added :AppArch and removed Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Mar 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Search Querying infrastructure in Kibana Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants