search

sunnyhub / thrift-protobuf-compare

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

ProtobufSerializer skips optional fields #9

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
ProtobufSerializer doesn't set values for many fields. This causes those
fields to not be serialized, resulting in protobuf executing more quickly
and producing fewer bytes. It could be argued this is a protobuf feature,
however the test has many fields that are left at their default value and
makes for misleading results. Besides, in real usage there will usually be
little or no fields at the default values.

Original issue reported on code.google.com by nathan.s...@gmail.com on 29 Sep 2009 at 4:01

GoogleCodeExporter commented 8 years ago 
Initialized all the fields, in SVN 83.

Original comment by nathan.s...@gmail.com on 29 Sep 2009 at 4:09