search

vert-x / mod-lang-clojure

Vert.x 2.x is deprecated - use instead
http://vertx.io
Other
74 stars 15 forks source link

Use Json(Object|Array) directly instead of serializing to json twice #111

Closed p14n closed 10 years ago

p14n commented 10 years ago

Now capable of encoding Datomic EntityMaps. Slight performance improvement from skipping the clojure.data.json step.

re: https://groups.google.com/forum/#!topic/vertx/OZhE0laA9I4

tobias commented 10 years ago

Great, thanks! I built the 1.0.0.Beta3 release last night just before your PR, so I'll merge this in and try to put out a Beta4 next week.

Apologies if I've already asked you do to this, but have you signed/can you sign the Eclipse CLA? I'll need that done before I can merge: http://www.eclipse.org/legal/CLA.php

p14n commented 10 years ago

Cool, ta. I've signed it previously for other vert commits.

tobias commented 10 years ago

I've merged this in, and cleaned up some of the functions a bit. I'm going to release a 1.0.0.RC1 with these changes today.