Skip to content

fix: do not panic on invalid multiple subqueries#26143

Merged
davidby-influx merged 1 commit intomaster-1.xfrom
DSB_multi_select
Mar 14, 2025
Merged

fix: do not panic on invalid multiple subqueries#26143
davidby-influx merged 1 commit intomaster-1.xfrom
DSB_multi_select

Conversation

@davidby-influx
Copy link
Copy Markdown
Contributor

Multiple subqueries in a FROM clause caused a
panic, insead of returning an error because
they are syntactically invalid. This corrects
that problem

closes #26139

Multiple subqueries in a FROM clause caused a
panic, insead of returning an error because
they are syntactically invalid. This corrects
that problem

closes #26139
@davidby-influx davidby-influx added area/queries area/influxql Issues related to InfluxQL query language kind/bug team/edge labels Mar 14, 2025
@davidby-influx davidby-influx self-assigned this Mar 14, 2025
@davidby-influx davidby-influx linked an issue Mar 14, 2025 that may be closed by this pull request
Copy link
Copy Markdown

@devanbenz devanbenz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread query/select_test.go
},
now: mustParseTime("1970-01-01T00:02:30Z"),
},
{
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding a test!

@davidby-influx davidby-influx merged commit 9e00f0d into master-1.x Mar 14, 2025
@davidby-influx davidby-influx deleted the DSB_multi_select branch March 14, 2025 20:39
devanbenz pushed a commit that referenced this pull request Mar 18, 2025
Multiple subqueries in a FROM clause caused a
panic, insead of returning an error because
they are syntactically invalid. This corrects
that problem

closes #26139

(cherry picked from commit 9e00f0d)
devanbenz added a commit that referenced this pull request Mar 19, 2025
Multiple subqueries in a FROM clause caused a
panic, insead of returning an error because
they are syntactically invalid. This corrects
that problem

closes #26139

(cherry picked from commit 9e00f0d)

Co-authored-by: davidby-influx <72418212+davidby-influx@users.noreply.github.com>
devanbenz added a commit that referenced this pull request May 9, 2025
Multiple subqueries in a FROM clause caused a
panic, insead of returning an error because
they are syntactically invalid. This corrects
that problem

closes #26139

(cherry picked from commit 9e00f0d)

Co-authored-by: davidby-influx <72418212+davidby-influx@users.noreply.github.com>
(cherry picked from commit e862501)
devanbenz added a commit that referenced this pull request May 12, 2025
Multiple subqueries in a FROM clause caused a
panic, insead of returning an error because
they are syntactically invalid. This corrects
that problem

closes #26139

(cherry picked from commit 9e00f0d)

Co-authored-by: davidby-influx <72418212+davidby-influx@users.noreply.github.com>
(cherry picked from commit e862501)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/influxql Issues related to InfluxQL query language area/queries kind/bug team/edge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Multiple SELECT clauses in FROM cause panic

2 participants