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

Kryo serialization fails(服务既是生产者又是消费者出现问题) #2881

Open
llx206 opened this issue Dec 4, 2018 · 7 comments

Comments

Projects
None yet
3 participants
@llx206
Copy link

commented Dec 4, 2018

#2179 (comment)

传输的数据较大,想使用kryo序列化,来提高性能。请问,在最新的2.6.5版本下,还是会出现
image

请问大神们,这个问题要如何解决呢,谢谢!!!!

@eoc2015

This comment has been minimized.

Copy link

commented Dec 5, 2018

感觉这个可以参考:https://www.2cto.com/kf/201805/743957.html

@llx206

This comment has been minimized.

Copy link
Author

commented Dec 6, 2018

@eoc2015 ,是要修改dubbo的代码,才能解决吗?

@cvictory

This comment has been minimized.

Copy link
Contributor

commented Dec 12, 2018

@cvictory

This comment has been minimized.

Copy link
Contributor

commented Dec 13, 2018

cvictory added a commit to cvictory/incubator-dubbo that referenced this issue Dec 14, 2018

@cvictory cvictory referenced this issue Dec 14, 2018

Open

modify kryo serialization. fix #2881 #2964

1 of 6 tasks complete
@llx206

This comment has been minimized.

Copy link
Author

commented Dec 17, 2018

@cvictory ,非常感谢你细致的回复。还有两个问题再请教下。
1.不要启用这个优化,也就是不要设置optimizer。
问:不启用优化,相比hessian2序列化协议,kryo的性能如何呢?
2.dubbo代码进行了初步的优化,进行了初步的测试,可以运行
问:针对部分接口维度的优化,新建一个ID和class列表,这个会在dubbo的哪个版本加入呢?

期待你的再次回复,谢谢!!

@cvictory

This comment has been minimized.

Copy link
Contributor

commented Dec 24, 2018

抱歉,上周休假了,网络实在是惨不忍睹。
问题1,我们没做过测试,部分数据来源于网络:https://x-rip.iteye.com/blog/1555293
同时强调一点,如果不用注册方式的话,尽量让报名和类名短,来自于:https://github.com/EsotericSoftware/kryo#optional-registration

问题2,预计要到2.6.6版本中支持。 我还在考虑是否可以考虑使用一种比较快速的方式来解决。 你可以在上面看到我提交的pr。


Sorry, the vacation last week, the network is really terrible.
Question 1, we have not tested, some of the data comes from the network: https://x-rip.iteye.com/blog/1555293.
At the same time, it is emphasized that if you do not need to register, try to make the registration and class name short, from: https://github.com/EsotericSoftware/kryo#optional-registration.

Question 2 is expected to be supported in version 2.6.6. I am still considering whether I can consider using a faster way to solve it. You can see the pr I submitted above.

@llx206

This comment has been minimized.

Copy link
Author

commented Dec 24, 2018

@cvictory 非常感谢你的再次解答。同时,也非常期待你们的新版本能用更快速的方式实现,期待新版本。谢谢!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.