marsevilspirit
marsevilspirit
check ci error.
We need all CI to pass to ensure code quality.
> > 这 reset 函数怎么没有一个地方调用？ > > The user should call the Reset method when retrieving a URL structure from the pool. You mean that users should handle this manually?...
Amazing work.
I have been quite busy recently and might only be able to review this PR the day after tomorrow.
Amazing code, I like your coding style. But we don't want to send attachments through a filter. Can you implement sending attachments through context?
> 我建议还是采用同样的机制。但是我没测试过。 tri.AppendToOutgoingContext(ctx, k, val) tri.FromIncomingContext(ctx) 这两个函数太低层了，我们不想暴露太低层的函数 如果要使用这两个函数的话，用户需要 ```go import "dubbo.apache.org/dubbo-go/v3/protocol/triple/triple_protocol" ``` 我还不想让用户直接import triple_protocol模块，而triple模块可以接受，或许可以在triple模块加入attachments的API？
> 如果需要我这边实现，可能要过两天我空点 对于attachments，我们想要合并一个完成度高的pr，而不是留作一个todo(因为todo不知道什么时候才会有人解决)，所以辛苦你了 如果最近太忙，也可以交给我接手
> 我看内部的代码，context还有个attanchmentkey，感觉是不是一个重复性的东西？而且我没试过，dubbogo走其他协议，还是否经过triple的代码，triple这一层感觉不适合，正常来讲，这套机制应该和协议不相干，是否还有更上层的接口。可以你接手，没问题的。 我一会看一下，单是使用AppendToOutgoingContext这样的函数应该是没问题的，因为只有triple协议用header来传attachments，我注意一下其他协议attachments的使用，设计一下如何实现更好。