search

taoensso / faraday

Amazon DynamoDB client for Clojure
https://www.taoensso.com/faraday
Eclipse Public License 1.0
238 stars 84 forks source link

Add support for the document API #103

Closed ricardojmendez closed 4 years ago

ricardojmendez commented 7 years ago

We don't yet support the "new" document API. Spinning this out from #91, see this comment for more details.

joelittlejohn commented 4 years ago

The Document API provides a much more pleasant interaction model for Java developers, however since we're already free to work with 'documents' in Clojureland (our edn literals) I think there's limited value in switching Faraday to the document API. Faraday is already giving developers an experience akin to the document API.

It could simplify some of the Faraday internals to use the Document API but would also carry a performance cost. If we decided to support both APIs (two different Faraday modes) we'd have a lot more code to maintain here.

I'm going to close this one but we're certainly open to ideas about why it might be good to revisit this.