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

query-frontend: Do not break scheduler connection on malformed queries #9786

Merged
merged 3 commits into from
Oct 31, 2024

Conversation

codesome
Copy link
Member

What this PR does

Instead of breaking the scheduler connection for malformed queries and eventually sending 500, instead send 400 and do not break the scheduler connection.

This is another version of what #9782 was trying to do.

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
@codesome codesome requested a review from a team as a code owner October 31, 2024 14:32
Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
@codesome codesome requested a review from francoposa October 31, 2024 15:13
@francoposa francoposa merged commit 587d234 into main Oct 31, 2024
29 checks passed
@francoposa francoposa deleted the codesome/frontendbug-take2 branch October 31, 2024 21:46
codesome added a commit that referenced this pull request Nov 1, 2024
#9786) (#9790)

* query-frontend: Do not break scheduler connection on malformed queries

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* Updated tests

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* lint

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

---------

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
(cherry picked from commit 587d234)

Co-authored-by: Ganesh Vernekar <ganeshvern@gmail.com>
codesome added a commit that referenced this pull request Nov 1, 2024
#9786) (#9788)

* query-frontend: Do not break scheduler connection on malformed queries

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* Updated tests

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* lint

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

---------

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
(cherry picked from commit 587d234)

Co-authored-by: Ganesh Vernekar <ganeshvern@gmail.com>
codesome added a commit that referenced this pull request Nov 1, 2024
#9786) (#9792)

* query-frontend: Do not break scheduler connection on malformed queries

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* Updated tests

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* lint

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

---------

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
(cherry picked from commit 587d234)

Co-authored-by: Ganesh Vernekar <ganeshvern@gmail.com>
codesome added a commit that referenced this pull request Nov 1, 2024
#9786) (#9789)

* query-frontend: Do not break scheduler connection on malformed queries

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* Updated tests

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* lint

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

---------

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
(cherry picked from commit 587d234)

Co-authored-by: Ganesh Vernekar <ganeshvern@gmail.com>
narqo added a commit that referenced this pull request Nov 6, 2024
…alformed queries (#9833)

* query-frontend: Do not break scheduler connection on malformed queries (#9786)

* query-frontend: Do not break scheduler connection on malformed queries

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* Updated tests

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* lint

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

---------

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
(cherry picked from commit 587d234)

* update changelog

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

---------

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>
Co-authored-by: Ganesh Vernekar <ganeshvern@gmail.com>
Co-authored-by: Vladimir Varankin <vladimir.varankin@grafana.com>
deniszh pushed a commit to deniszh/mimir that referenced this pull request Nov 14, 2024
grafana#9786)

* query-frontend: Do not break scheduler connection on malformed queries

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* Updated tests

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

* lint

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>

---------

Signed-off-by: Ganesh Vernekar <ganeshvern@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants