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

Bug Report: Query timeout applies only on Route and not on the entire query execution #16624

Open
GuptaManan100 opened this issue Aug 21, 2024 · 0 comments · May be fixed by #16619
Open

Bug Report: Query timeout applies only on Route and not on the entire query execution #16624

GuptaManan100 opened this issue Aug 21, 2024 · 0 comments · May be fixed by #16619

Comments

@GuptaManan100
Copy link
Member

Overview of the Issue

Currently, the query timeout expressed by the --query-timeout flag, query_timeout session variable and Query_timeout_ms comment directive only applies to the query execution on the routes and not the entire execution of the query. This is not the correct behavior, since if the query is running with a timeout of 3 seconds, and is running a join that can potentially run 100s of select queries each taking about a second. Ideally, this query should timeout after 3 seconds, but currently it would continue to run and would not timeout since we only add the 3 second timeout to the routes, and they all run under a second, even if the entire execution takes much longer.

Reproduction Steps

  1. Set up a cluster
  2. Run a query that has multiple joins with a timeout such that total execution time is more than the timeout.

Binary Version

main

Operating System and Environment details

-

Log Fragments

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant