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

[RCF] Serialize|Deserialize thrift message on demands (support protobuf's fieldmask) | 支持 thrift 按需序列化(支持 pb fieldmask) #1071

Open
AsterDY opened this issue Aug 14, 2023 Discussed in #1070 · 0 comments

Comments

@AsterDY
Copy link
Contributor

AsterDY commented Aug 14, 2023

Discussed in #1070

Originally posted by AsterDY August 14, 2023
Protobuffer provides an ability called FieldMask to serialize|deserialize message on demands. And Netflix has used it in practice to improve the performance of API services: https://netflixtechblog.com/practical-api-design-at-netflix-part-1-using-protobuf-fieldmask-35cfdc606518. As for thrift, there is no such a direct support -- we can define a trimmed mirror IDL for every client as a workaround, but it is unsuitable for large-scale application development that every client must main a specified code-gen product for the trimmed IDL. In contrast, It is a trend to use a common repo to maintain code-gen products nowadays, and under such a situation, the overhead of redundant fields will get higher and higher. Therefore, we need to propose a way to support fieldmask to achieve serializing|deserializing messages on demands.
Is anyone interested in it? If so, please let us know

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

No branches or pull requests

1 participant