DMwangnima opened a new issue, #2554: URL: https://github.com/apache/dubbo-go/issues/2554
<!-- Please only use this template for submitting enhancement requests --> **What would you like to be added**: Triple Open-Tracing support: 1. Consider a better way to inject Open-Tracing configurations to Triple protocol adaptation layer(/protocol/triple). For old Triple(/protocol/dubbo3), it retrieves configurations from config package directly. It is not a good design. 2. Implement Open-Tracing Interceptor for Triple protocol. You can refer to this part(https://connectrpc.com/docs/go/observability). But there are some differences between Triple protocol and connect-go. 3. Tests implementation. For Triple protocol adaptation part, Please design tests to verify that Open-Tracing configurations have been injected successfully and the necessary information in these configurations have been processed well. You can add tests in /protocol/triple/triple_test.go or some place more ideal. For Triple protocol part, It is a good chance to learn the details about this protocol from outside to inside. You should write some unit tests to verify that Open-Tracing Interceptor has been configured successfully and this functionality works well in the context of Triple protocol. For dubbo-go-samples(Integration test), please refer to /tracing directory and write new tests using new API. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
