search

elodina / go-avro

Apache Avro for Golang
http://elodina.github.io/go-avro/
Apache License 2.0
129 stars 55 forks source link

GenericDatum: Process default values correctly #36

Closed arunmk closed 9 years ago

arunmk commented 9 years ago

Currently:

  1. Default values are not processed well in GenericDatum. They are not set at all.
  2. If a number literal is used as a default value, it is converted internally to float64 due to GoLang JSON unmarshalling.

The above need to be fixed.

arunmk commented 9 years ago

This is fixed by: Pull: https://github.com/stealthly/go-avro/pull/37 Commit: https://github.com/arunmk/go-avro/commit/7eb45b451f12afb7602dbe40e462faf0d6ed52dc

The commit has tests updated.

serejja commented 9 years ago

Merged to master. Thanks