Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Apr 2, 2024. It is now read-only.

Conversation

@antekresic
Copy link
Member

@antekresic antekresic commented May 27, 2022

Description

When translating Jaeger traces to OTEL format, some Jaeger tags get
converted to the to OTEL specific fields and gets removed from the tag map. This
change takes this conversion into account when querying traces using these
tags.

Merge requirements

Please take into account the following non-code changes that you may need to make with your PR:

  • CHANGELOG entry for user-facing changes
  • Updated the relevant documentation

@antekresic antekresic requested a review from paulfantom as a code owner May 27, 2022 11:00
@antekresic antekresic self-assigned this May 27, 2022
@antekresic antekresic requested a review from a team as a code owner May 27, 2022 11:00
@antekresic antekresic force-pushed the fix-jaeger-translation branch 2 times, most recently from 5541be3 to 5a07a40 Compare May 31, 2022 08:57
@antekresic antekresic changed the title Fix Jaeger trace querying by status code tag Fix Jaeger trace querying by converted Jaeger tags May 31, 2022
@antekresic antekresic force-pushed the fix-jaeger-translation branch from 5a07a40 to ab555a9 Compare May 31, 2022 12:44
@antekresic antekresic force-pushed the fix-jaeger-translation branch from ab555a9 to 857401a Compare June 1, 2022 09:49
@antekresic antekresic requested a review from a team as a code owner June 1, 2022 09:49
@antekresic antekresic force-pushed the fix-jaeger-translation branch from 857401a to c489f9f Compare June 1, 2022 10:00
@antekresic antekresic force-pushed the fix-jaeger-translation branch from c489f9f to d95efde Compare June 1, 2022 14:12
@antekresic antekresic enabled auto-merge (rebase) June 1, 2022 14:22
When translating Jaeger traces to OTEL format, some Jaeger tags get
converted to the to OTEL specific fields and gets removed from the tag map. This
change takes this conversion into account when querying traces using these
tags.
@antekresic antekresic force-pushed the fix-jaeger-translation branch from d95efde to 5f765a1 Compare June 1, 2022 14:25
@antekresic antekresic merged commit 5c5dd5a into timescale:master Jun 1, 2022
@harkishen harkishen mentioned this pull request Jun 2, 2022
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants