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

[TraceQL] != nil is not symmetric #3989

Open
joe-elliott opened this issue Aug 21, 2024 · 1 comment · May be fixed by #3991
Open

[TraceQL] != nil is not symmetric #3989

joe-elliott opened this issue Aug 21, 2024 · 1 comment · May be fixed by #3991
Assignees
@electron0zero
Labels
keepalive Label to exempt Issues / PRs from stale workflow traceql type/bug Something isn't working

Comments

@joe-elliott
Copy link
Member

Describe the bug

!= nil is only evaluated correctly one way.

This returns results:

{ span.accept != nil }

This does not:

{ nil != span.accept }

The best fix would be to make it work both ways. If that is difficult for some reason we should at least throw an error.

image

image
@joe-elliott joe-elliott added type/bug Something isn't working traceql labels Aug 21, 2024
@electron0zero electron0zero self-assigned this Aug 21, 2024
@electron0zero electron0zero moved this from Todo to In Progress in Tempo squad Aug 21, 2024
@electron0zero electron0zero linked a pull request Aug 21, 2024 that will close this issue
Open
3 tasks
@github-actions GitHub Actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had any activity in the past 60 days.
The next time this stale check runs, the stale label will be removed if there is new activity. The issue will be closed after 15 days if there is no new activity.
Please apply keepalive label to exempt this Issue.

@github-actions github-actions bot added the stale Used for stale issues / PRs label Oct 21, 2024
@electron0zero electron0zero added keepalive Label to exempt Issues / PRs from stale workflow and removed stale Used for stale issues / PRs labels Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@electron0zero electron0zero

Labels
keepalive Label to exempt Issues / PRs from stale workflow traceql type/bug Something isn't working
Projects
Tempo squad
Status: Next
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

TraceQL: make nil type symmetric electron0zero/tempo
2 participants
@joe-elliott @electron0zero