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

Remove the generic_hessian_write_definition_everytime parameter and write definition as needed #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

wangjianxin199003
Copy link

  • 通过对com.alipay.hessian.generic.io.GenericObjectSerializer.ObjectDefinition进行比较来决定当前要写入的GenericObject是否要重新写入definition
  • 无需对每一个GenericObject都写入definition
  • 减小序列化体积,节省带宽

@sofastack-bot sofastack-bot bot changed the title 去除generic_hessian_write_definition_everytime参数,按需进行definition写入 Remove the generic_hessian_write_definition_everytime parameter and write definition as needed Jul 8, 2020
@sofastack-bot
Copy link

sofastack-bot bot commented Jul 8, 2020

Hi @wangjianxin199003, we detect non-English characters in the issue. This comment is an auto translation by @sofastack-robot to help other users to understand this issue.

We encourage you to describe your issue in English which is more friendly to other users.

-By comparing com.alipay.hessian.generic.io.GenericObjectSerializer.ObjectDefinition to determine whether the current GenericObject to be written should be rewritten to definition-No need to write definition for every GenericObject-Reduce serialization volume and save bandwidth

@sofastack-bot
Copy link

sofastack-bot bot commented Jul 8, 2020

Hi @wangjianxin199003, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

@OrezzerO
Copy link
Contributor

It's a good idea

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

Successfully merging this pull request may close these issues.

None yet

2 participants