Skip to content

support header namespace isolation#260

Closed
pingliu wants to merge 1 commit into
apache:masterfrom
pingliu:master
Closed

support header namespace isolation#260
pingliu wants to merge 1 commit into
apache:masterfrom
pingliu:master

Conversation

@pingliu

@pingliu pingliu commented Dec 15, 2022

Copy link
Copy Markdown

[] support header namespace isolation

@wu-sheng

Copy link
Copy Markdown
Member

This had been removed from the protocol months ago. We don't support this anymore.

Ref apache/skywalking-java#123

Change log

  • [Breaking Change] Remove the namespace from cross process propagation key.

The points we did are

  1. This is never defined in the protocol/APIs.
  2. Breaking a distributed tracing seems unreasonable because they are really accessing each other.
  3. In four years of history, there is no other agent tried to support this, and we think <1> and few use cases are the reasons.

@pingliu

pingliu commented Dec 15, 2022

Copy link
Copy Markdown
Author

ok

@kezhenxu94 kezhenxu94 closed this Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants