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

SEARCH-8101 Cancel the query upon timeout #31

Merged

Conversation

dcheckoway
Copy link
Collaborator

@dcheckoway dcheckoway commented Dec 10, 2024

If the query takes longer than the configured timeout, at least try to cancel the search job on the Cribl side.

To test this, you can set the timeout to a relatively low value like 5 or 10 seconds, then run a query like

dataset="$vt_dummy" second < 420

On the Cribl side, you can find the job by its id and confirm that it was indeed canceled.

@dcheckoway dcheckoway force-pushed the dc/SEARCH-8103-grafana-plugin-include-info-about-scheduled-queries-in-the-timeout-error branch from 7f25a8a to 98c9c07 Compare December 10, 2024 20:31
Base automatically changed from dc/SEARCH-8103-grafana-plugin-include-info-about-scheduled-queries-in-the-timeout-error to dc/v1.1.2 December 10, 2024 20:35
@dcheckoway dcheckoway force-pushed the dc/SEARCH-8101-grafana-plugin-cancel-query-upon-timeout branch from 4ef78d0 to 1ecea93 Compare December 10, 2024 20:36
@dcheckoway dcheckoway merged commit 7fdcda1 into dc/v1.1.2 Dec 10, 2024
2 checks passed
@dcheckoway dcheckoway deleted the dc/SEARCH-8101-grafana-plugin-cancel-query-upon-timeout branch December 10, 2024 20:37
dcheckoway added a commit that referenced this pull request Dec 12, 2024
* go get -u github.com/grafana/grafana-plugin-sdk-go

* npx -y @grafana/create-plugin@latest update

* SEARCH-8100 Make query timeout configurable (#29)

* SEARCH-8103 Include info about scheduled queries in the timeout error (#30)

* SEARCH-8309 Allow savedSearchId to be composed using variable(s) (#27)

* SEARCH-8309 Allow savedSearchId to be composed using variable(s)

* SEARCH-8309 Update QueryEditor to allow typing savedSearchId with ${variable}

* SEARCH-8101 Cancel the query upon timeout (#31)

* ran "npx -y @grafana/create-plugin@latest update" again

* Bumps for v1.1.2
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