Netflix / PigPen

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

Avro Loader #101

Closed theJohnnyBrown closed 9 years ago

theJohnnyBrown commented 9 years ago

TODO

mbossenbroek commented 9 years ago

Looks good! I'm looking forward to seeing the sample data & tests. I think that'll make it a lot more clear how the schema is being used.

theJohnnyBrown commented 9 years ago

It would be great to hear about the gradle/clojure workflow people are using on this project - does anyone know how to get a repl with the dependencies loaded? I've resorted to creating another leningen project, with a dependency on pigpen-avro, opening core.clj in a cider buffer and eval-ing it in that repl.

EDIT: there's ../gradlew clojureRepl but that seems to have issues for me.

theJohnnyBrown commented 9 years ago

@mbossenbroek, I think the issues you mentioned have been addressed. I'll save the storage side for next time (Or for someone else!). Anything else you need to see?

mbossenbroek commented 9 years ago

Looks great! Just a couple of polish items. Thanks for the work!

mbossenbroek commented 9 years ago

Almost forgot - what issues are you seeing with ./gradlew clojureRepl? IIRC, I'm getting that from clojuresque and it should work. Personally, I use eclipse and CCW - one keystroke and I have a repl in my IDE. IntelliJ+Cursive should work similarly.

theJohnnyBrown commented 9 years ago

Here's the output from running clojureRepl: https://gist.github.com/theJohnnyBrown/faa5782176ebe327dade

theJohnnyBrown commented 9 years ago

updated

mbossenbroek commented 9 years ago

Released in 0.2.13