Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

使用protostuff 序列化时候 心跳 WrapperUtils出错 #4568

Closed
2 tasks done
chenyu2016 opened this issue Jul 15, 2019 · 5 comments
Closed
2 tasks done

使用protostuff 序列化时候 心跳 WrapperUtils出错 #4568

chenyu2016 opened this issue Jul 15, 2019 · 5 comments

Comments

@chenyu2016
Copy link

chenyu2016 commented Jul 15, 2019

  • I have searched the issues of this repository and believe that this is not a duplicate.
  • I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

  • Dubbo version: 2.7.1
  • Operating System version: mac
  • Java version: 1.8

Steps to reproduce this issue

  1. protocol设置为dubbo 序列化设置为 protosubff
  2. 等待心跳

Pls. provide [GitHub address] to reproduce this issue.

Expected Result

What do you expected from the above steps?

Actual Result

WrapperUtils.needWrapper(Object obj){
return needWrapper(obj.getClass()); //这里obj 是null报错
}
因为NettyClientHandler.userEventTriggered()
方法创建Request对象时候 mData 属性没有null
请问是不是我使用protosubff 姿势不对

Just put your stack trace here!
@cvictory
Copy link
Contributor

I find there is no sample in https://github.com/apache/dubbo-samples.
Can you supply a protostuff sample and post a pull request in https://github.com/apache/dubbo-samples.

@chenyu2016
Copy link
Author

我发现https://github.com/apache/dubbo-samples中没有样本。
您可以在https://github.com/apache/dubbo-samples中提供protostuff示例并发布拉取请求。

好的 一会我提交一个 请求
并且 我还发现 @reference注解创建消费者时 MethodConfig 对@method标签和使用xml创建 对 onreturn
onthrow oninvoke 这三个属性的即使不一致

@beiwei30
Copy link
Member

apache/dubbo-samples#104

@beiwei30
Copy link
Member

#4645

@beiwei30 beiwei30 added this to the 2.7.4 milestone Aug 1, 2019
@cvictory cvictory removed this from the 2.7.4 milestone Sep 4, 2019
@cvictory cvictory closed this as completed Sep 4, 2019
@hanlukk
Copy link

hanlukk commented Dec 28, 2020

这个问题解决了吗?哪个版本解决了?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants