-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 with math returns field with no name #5730
Comments
I found where the error is coming from. What should the column name be where there are multiple fields in the binary expression? |
For something like?
Maybe concatenate the fields w/ |
Sure! Should we remove duplicates? If we have |
I would probably go with |
I'll go with |
The name of the column will be every measurement located inside of the math expression in the order they are encountered in within the expression. Fixes #5730.
The name of the column will be every measurement located inside of the math expression in the order they are encountered in within the expression. Also handle `*influxql.ParenExpr` in the function `(*influxql.Field).Name()` Fixes #5730.
Using math on a field returns a field with an empty name. If this used used in a continuous query or select into, it the points can't be inserted and causes some of the issues seen in #5697.
To repro:
Notice the second example is missing a
value
column header.Selecting multiple columns doesn't fix it.
Using an
as
alias does work though:The text was updated successfully, but these errors were encountered: