Skip to content

Fix autocomplete returning erroneous values - #3339

Merged
joe-elliott merged 7 commits into
grafana:mainfrom
joe-elliott:fix-tag-values
Jan 29, 2024
Merged

Fix autocomplete returning erroneous values#3339
joe-elliott merged 7 commits into
grafana:mainfrom
joe-elliott:fix-tag-values

Conversation

@joe-elliott

@joe-elliott joe-elliott commented Jan 26, 2024

Copy link
Copy Markdown
Collaborator

What this PR does:
In certain circumstances filtered autocomplete would return incorrect results due to an incorrect iterator scope. The actual fix is this line

  • removed an unused param in createDistinctAttributeOperator
  • added integration tests for filtered autocomplete

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]
@joe-elliott
joe-elliott marked this pull request as ready for review January 26, 2024 18:18
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>

@mapno mapno left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Nice catch

@joe-elliott
joe-elliott merged commit 870b367 into grafana:main Jan 29, 2024
mattdurham pushed a commit to mattdurham/tempo that referenced this pull request Jun 18, 2026
* correct definition level

Signed-off-by: Joe Elliott <number101010@gmail.com>

* removed unused param

Signed-off-by: Joe Elliott <number101010@gmail.com>

* Add autocomplete test

Signed-off-by: Joe Elliott <number101010@gmail.com>

* span unscoped

Signed-off-by: Joe Elliott <number101010@gmail.com>

* changelog

Signed-off-by: Joe Elliott <number101010@gmail.com>

* added multiple conditions and commented out unscoped test

Signed-off-by: Joe Elliott <number101010@gmail.com>

* lint

Signed-off-by: Joe Elliott <number101010@gmail.com>

---------

Signed-off-by: Joe Elliott <number101010@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants