Upgrade thrift to 0.9.3

eishay / jvm-serializers

Benchmark comparing serialization libraries on the JVM

http://groups.google.com/group/java-serialization-benchmarking

3.29k stars 562 forks source link

Upgrade thrift to 0.9.3 #70

Closed fishercoder1534 closed 7 years ago

fishercoder1534 commented 7 years ago

Upgrade Thrift to the latest version today: 0.9.3

Compiled and ran the test, all worked fine.

cakoose commented 7 years ago

I noticed you updated the generated code in "tpc/pregen/media.thrift", which is the correct thing to update. I also notice some additional generated files at "tpc/gen-java" -- what's that for?

Also, can you squash everything into a single commit?

fishercoder1534 commented 7 years ago

Thanks a lot @cakoose for your advice, I've deleted my old fork and forked a new one and squashed everything into one single commit, also thanks for spotting those unnecessary files under "tpc/gen-java", I've cleaned them up. I'll close this Pull Request and open a new one for the same regard. Thanks again! :)