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.

Sign up for GitHub

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

Jump to bottom

Prevent a panic when aggregates are used in an inner query with a raw query #7966

Merged
merged 1 commit into from
Feb 8, 2017
Merged

Prevent a panic when aggregates are used in an inner query with a raw query #7966

merged 1 commit into from
Feb 8, 2017

Commits on Feb 8, 2017

  1. Prevent a panic when aggregates are used in an inner query with a raw… 

    … query

The following types of queries will panic:

    SELECT mean, host FROM (SELECT mean(value) FROM cpu GROUP BY host)
    SELECT top(sum, host, 3) FROM (SELECT sum(value) FROM cpu GROUP BY host)

These queries _should_ work, but due to a current limitation with
aggregate functions, the aggregate functions won't return any auxiliary
fields. So even if a tag is not an auxiliary field, it is treated that
way by the query engine and this query will fail.

Fixing this properly will take a longer period of time. This fix just
prevents the panic from killing the server while we fix this for real.
    @jsternberg
    jsternberg committed Feb 8, 2017
    Configuration menu
    Copy the full SHA
    2ad1668 View commit details
    Browse the repository at this point in the history