chenBright merged PR #2948:
URL: https://github.com/apache/brpc/pull/2948
--
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: dev-unsubscr...@brpc.apache.or
wwbmmm commented on PR #2948:
URL: https://github.com/apache/brpc/pull/2948#issuecomment-2799938172
> 我觉得tag是一个上下文，在系统启动过程应该初始化好，后续的执行流总是切换tag感觉就不太合理了。
有这样的场景，从tag1接受网络消息，然后用户将任务丢给tag2处理，然后在tag2里面返回网络应答，这个过程是不会创建新socket的，接受网络消息的socket是tag1，在它的生命周期里面tag不会改变。
bthread的创建是用户可调用、可配置的，但是socket的创
yanglimingcn commented on PR #2948:
URL: https://github.com/apache/brpc/pull/2948#issuecomment-2798723130
我觉得tag是一个上下文，在系统启动过程应该初始化好，后续的执行流总是切换tag感觉就不太合理了。
有这样的场景，从tag1接受网络消息，然后用户将任务丢给tag2处理，然后在tag2里面返回网络应答，这个过程是不会创建新tag的，接受网络消息的socket是tag1，在它的生命周期里面tag不会改变。
bthread的创建是用户可调用、可配置的，但是soc
wwbmmm commented on PR #2948:
URL: https://github.com/apache/brpc/pull/2948#issuecomment-2798717061
按照这个逻辑：https://github.com/apache/brpc/blob/master/src/bthread/bthread.cpp#L266
如果初始值设置成BTHREAD_TAG_INVALID，默认就是在当前的tag中运行
如果初始值设置成bthread_self_tag()的话，我担心有可能初始化SocketOptions和实际使用Socket的不
yanglimingcn commented on PR #2948:
URL: https://github.com/apache/brpc/pull/2948#issuecomment-2798624971
> 这样会有会有某些本来需要使用BTHREAD_TAG_DEFAULT的地方误用了bthread_self_tag()呢？
如果担心漏掉是否可以把初始值设置成BTHREAD_TAG_INVALID，然后在每个用到的地方显式设置？
bthread_self_tag正常会返回default，只要主动设置了其它的tag的上下文，在这个tag的上下文执行，才会设
wwbmmm commented on PR #2948:
URL: https://github.com/apache/brpc/pull/2948#issuecomment-2798467802
这样会有会有某些本来需要使用BTHREAD_TAG_DEFAULT的地方误用了bthread_self_tag()呢？
如果担心漏掉是否可以把初始值设置成BTHREAD_TAG_INVALID，然后在每个用到的地方显式设置？
--
This is an automated message from the Apache Git Service.
To respond to
