MemoRyAxis / thrift-protobuf-compare

Automatically exported from code.google.com/p/thrift-protobuf-compare
0 stars 0 forks source link

total time column measures instance creation twice #17

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
The "total time" column is implemented with

timeCreate + timeSerializeDifferentObjects + timeDeserializeNoFieldAccess

in trunk. Since timeSerializeDifferentObjects already contains calls to
serializer.create() we measure instance creation twice.

This is probably the reason, why avro, which is slow in instance creation,
was faster in v1 than in trunk...

Original issue reported on code.google.com by michael....@gmail.com on 6 Jan 2010 at 9:55

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Fixed in trunk

Original comment by michael....@gmail.com on 6 Jan 2010 at 9:57