Skip to content

Conversation

@Civil
Copy link
Member

@Civil Civil commented Jun 22, 2022

Use vtproto@0.3.0 and protobuf@1.28.0 for generated files

Also enable all features that vtproto provides

Use vtproto@0.3.0 and protobuf@1.28.0 for generated files

Also enable all features that vtproto provides
@Civil Civil requested a review from bom-d-van June 22, 2022 09:13
@Civil
Copy link
Member Author

Civil commented Jun 22, 2022

That's what I've talked about in the issue for go-carbon. Since the initial release and generation, vtproto implemented a bunch of new stuff, like reflect-less UnmarshalVT functions, Custom object Pool, etc. that seems to be extremely useful and should dramatically increase performance compared to what was in v2 tag.

@Civil Civil requested a review from deniszh June 22, 2022 09:14
@bom-d-van
Copy link
Member

thanks, I will try to test it on go-carbon sometime next week on prod.

@Civil Civil merged commit 7bb0ad7 into master Jun 22, 2022
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

Successfully merging this pull request may close these issues.

3 participants