search

Netflix / PigPen

Map-Reduce for Clojure
Apache License 2.0
565 stars 55 forks source link

Implement native->clojure for GenericData$EnumSymbol #135

Closed ljank closed 9 years ago

ljank commented 9 years ago

We're using enums in Avro schemas and reading data files fails while on missing native->clojure for GenericData$EnumSymbol.

I know I shouldn't create PR based on master, but I couldn't find appropriate branch for this tiny change :)

mbossenbroek commented 9 years ago

Master is correct for avro work. I haven't merged it to the cascading branch yet.

@theJohnnyBrown wrote the avro loader - it looks good to me, but just want to verify with him as well.

theJohnnyBrown commented 9 years ago

LGTM, merge away. @ljank thanks especially for including tests