search

RameshRaj / thrift-protobuf-compare

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

total time column misleading for CheckingObjectSerializer #18

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
The "total time" column always includes timeDeserializeNoFieldAccess as
value for deserialization. 

CheckingObjectSerializers were introduced to show how lazy deserializers
perform in the case, that
 - no fields are accessed
 - only topmost fields are accessed
 - all fields are accessed
on the deserialized instances.

To make a fair comparison with "eager" deserializers, we should calculate
"total time" by using timeDeserializeAndCheckAllFields instead of
timeDeserializeNoFieldAccess for CheckingObjectSerializers.

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

GoogleCodeExporter commented 8 years ago 
fixed in trunk

Original comment by michael....@gmail.com on 6 Jan 2010 at 10:04