dmsolr opened a new pull request #5644: URL: https://github.com/apache/skywalking/pull/5644
Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bug fix - [x] New feature provided - [ ] Improve performance - Related issues #1534 ___ ### New feature or improvement ## Provide Thrift Plugin Because Thrift cannot carry `metadata` to transport Trace Header in the original API, we transport those by wrapping `TProtocolFactory` to do that. ### Principle about transport Trace Header Thrift allows us to append any additional field in the `Message` even if the receiver doesn't deal with them. This data is going to be skipped while no one reads. Base on this, we take the 8888th field of Message to store Trace Header(or metadata) and to transport. That means the message with 8888 [field id](http://thrift.apache.org/docs/idl#field-id) must be revered. Anyway, I don't think have any message big in reality. ### Snapshot TODO ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
