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: Add array attribute syntax and logic to detect arrays #4034

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

TraceQL: Add array attribute syntax and logic to detect arrays #4034

wants to merge 6 commits into from

Conversation

electron0zero
Copy link
Member

What this PR does:

adds support for [] support in traceQL for strict array only matches??

<TODO: add more details with examples>

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@electron0zero electron0zero changed the title TraceQL: Add array attribute syntax and logic to detect array TraceQL: Add array attribute syntax and logic to detect arrays Aug 29, 2024
@mdisibio mdisibio mentioned this pull request Sep 20, 2024
Open
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 4, 2024

This PR 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. This pull request will be closed in 15 days if there is no new activity.
Please apply keepalive label to exempt this Pull Request.

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

@joe-elliott joe-elliott Awaiting requested review from joe-elliott joe-elliott will be requested when the pull request is marked ready for review joe-elliott is a code owner

@mdisibio mdisibio Awaiting requested review from mdisibio mdisibio will be requested when the pull request is marked ready for review mdisibio is a code owner

@mapno mapno Awaiting requested review from mapno mapno will be requested when the pull request is marked ready for review mapno is a code owner

@yvrhdn yvrhdn Awaiting requested review from yvrhdn yvrhdn will be requested when the pull request is marked ready for review yvrhdn is a code owner

@zalegrala zalegrala Awaiting requested review from zalegrala zalegrala will be requested when the pull request is marked ready for review zalegrala is a code owner

@ie-pham ie-pham Awaiting requested review from ie-pham ie-pham will be requested when the pull request is marked ready for review ie-pham is a code owner

@stoewer stoewer Awaiting requested review from stoewer stoewer will be requested when the pull request is marked ready for review stoewer is a code owner

@javiermolinar javiermolinar Awaiting requested review from javiermolinar javiermolinar will be requested when the pull request is marked ready for review javiermolinar is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
keepalive Label to exempt Issues / PRs from stale workflow
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@electron0zero