Skip to content

[bugfix] Remove extra primary iter#3903

Merged
ie-pham merged 2 commits intografana:mainfrom
ie-pham:panic
Jul 24, 2024
Merged

[bugfix] Remove extra primary iter#3903
ie-pham merged 2 commits intografana:mainfrom
ie-pham:panic

Conversation

@ie-pham
Copy link
Copy Markdown
Contributor

@ie-pham ie-pham commented Jul 24, 2024

What this PR does: Primary iter is appended twice before. This fixes the issue in query range group by event attributes.

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]

Copy link
Copy Markdown
Contributor

@mapno mapno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Do we need a changelog entry? Is the changes in #3847 needed now?

@ie-pham ie-pham merged commit c27bf8b into grafana:main Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants