Add prost benchmarks - Githubissues

tafia / quick-protobuf

A rust implementation of protobuf parser

MIT License

452 stars 87 forks source link

Add prost benchmarks #63

Closed overvenus closed 7 years ago

overvenus commented 7 years ago

Hello,

There is a new protobuf crate Prost. I think it would be a good idea to benchmark it too. :-)

This PR does:

update rust-protobuf.
update generated files by latest rust-protobuf and quick-protobuf
add prost benchmarks.

TBD:

[x] https://github.com/danburkert/prost/issues/17
[x] dead_code warning in perftest_data_prost.rs danburkert/prost#19
[x] updating benchmark result.
[x] update perftest_data_quick.rs #59

tafia commented 7 years ago

Awesome Thanks!

tafia commented 7 years ago

I just merged #59, about codegen improvement. There should be no difference performance wise but still the codegen may be different. Do you mind rebasing your PR in case? Thanks

overvenus commented 7 years ago

@tafia No problem! This PR is WIP, when I think it's ready, I will rebase and squash into one commit. :grinning:

overvenus commented 7 years ago

Hello @tafia I think its ready, could you have a look?

tafia commented 7 years ago

I'll have a look this week. I don't have time for the moment. Thanks a lot!