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

[TraceQL] Support range operators for strings #2181

Closed
joe-elliott opened this issue Mar 8, 2023 · 0 comments · Fixed by #2321
Closed

[TraceQL] Support range operators for strings #2181

joe-elliott opened this issue Mar 8, 2023 · 0 comments · Fixed by #2321
Assignees
Labels

Comments

@joe-elliott
Copy link
Member

Is your feature request related to a problem? Please describe.
This query:

{span.http.status_code >= "200"}

will fail with this error:

illegal operation for the given types: span.http.status_code >= `200`

Support range operators (>, >= ,<=, <) for strings. Note that this should not attempt to convert strings to number values, but should instead use lexicographic ordering when comparing strings.

@joe-elliott joe-elliott moved this from Todo to Next in Tempo squad Mar 8, 2023
@ie-pham ie-pham self-assigned this Apr 4, 2023
@ie-pham ie-pham moved this from Next to In Progress in Tempo squad Apr 5, 2023
@ie-pham ie-pham moved this from In Progress to In Review in Tempo squad Apr 10, 2023
@ie-pham ie-pham moved this from In Review to In Progress in Tempo squad Apr 11, 2023
@ie-pham ie-pham moved this from In Progress to Next in Tempo squad Apr 11, 2023
@ie-pham ie-pham moved this from Next to In Progress in Tempo squad Apr 11, 2023
@ie-pham ie-pham moved this from In Progress to In Review in Tempo squad Apr 19, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in Tempo squad Apr 21, 2023
@ie-pham ie-pham moved this from Done to Archived in Tempo squad Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants