Tsukikage7 commented on issue #3568: URL: https://github.com/apache/dubbo-go/issues/3568#issuecomment-5131195204
@AlexStocks @Modo-me, I have posted a narrowed ownership proposal under #3562: https://github.com/apache/dubbo-go/issues/3562#issuecomment-5131193627 I do not intend to take or duplicate the filter auto-injection, default-on behavior, logging-default changes, probe defaults, or OpenTracing cleanup proposed here. The only possible overlap is Dubbo protocol error classification. I have proposed owning only a protocol-neutral shared classification core, subject to maintainer and current-contributor confirmation. I will not start that implementation until the boundary is agreed. If either of you is already implementing the classification itself, please let me know and I will leave it in your scope. If a split is useful, my proposed boundary is: - #3562: shared typed classifier contract/core, preserving causes and using `errors.Is`/`errors.As`; - #3568: auto-injection/default wiring and the agreed adapter/integration work. I want to make sure we coordinate before any code is written and do not open parallel PRs touching the same behavior. -- 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]
