http://dev.clojure.org/jira This file is an XML representation of an issue en-us 4.4 649 25-07-2011 http://dev.clojure.org/jira/browse/JDBC-3 java.jdbc <p>Stuart Sierra: In general, c.c.sql is inconsistent about using tuples of column values versus maps of column=&gt;value pairs. Both are useful, and I'd like to see versions of all the core functions that support both modes.</p> JDBC-3

Inconsistency between tuples and regular maps

Task Minor Resolved Duplicate Sean Corfield Sean Corfield Sun, 8 May 2011 21:15:04 -0500 Wed, 12 Oct 2011 16:29:40 -0500 Wed, 12 Oct 2011 16:29:40 -0500 0 0 <p>Having reviewed this after using the library for quite a while, I don't really see any inconsistencies except, perhaps in the naming of update-values (and update-or-insert-values).</p> <p>insert-values is the base function, taking a table name, a vector of column names and any number of vectors of column values. Generates one insert statement (containing multiple rows). If only one row is inserted, returns a map of generated keys if the database supports that.</p> <p>insert-rows takes a table name and any number of vectors of column values (for all columns); uses insert-values.</p> <p>insert-records takes a table name and any number of maps (containing column/value pairs); uses insert-values to insert one record at a time (and thus may return a sequence of maps of generated keys).</p> <p>insert-record takes a table name and a single map; uses insert-records and returns a map of generated key if the database supports that.</p> <p>delete-rows takes a table name and a where clause. Perhaps delete-where might be a better name?</p> <p>update-values takes a table name, a where clause and a map (of column/value pairs to update). Arguably this should be renamed since it doesn't take the same argument pattern as insert-values, but whilst update-records might sound more appropriate it only takes one "record" and update-record is clearly not right since it can update more than one row!</p> <p>update-or-insert-values (taking the same arguments as update-values) can however lead to inserting a single record but it can also update multiple records instead.</p> <p>On both of the latter functions, an analogy could be drawn with the findAndModify command in MongoDB which has an upsert option (to insert the document if no matches are found for the update), but most notably, findAndModify only updates a single record. I don't believe SQL has any portable equivalent but perhaps find-and-modify is a better name for update-values? Or perhaps, in the spirit of my suggestion for delete-where, we might call it update-where - and use an optional keyword argument to indicate that an insert-record should be performed when no rows match?</p> <p>The only other comment I could make on update-values is that it might be nice to have a variant that took: table where-params column-names value-group - somewhat inline with insert-values (although not allowing any number of vectors of column values).</p> <p>The API aspects of this ticket are covered in <a href="http://dev.clojure.org/jira/browse/JDBC-20" title="Create new clean API that accepts connection etc (post 0.1.0)"><del>JDBC-20</del></a> (post 0.1.0) and the struct-map issue is covered in <a href="http://dev.clojure.org/jira/browse/JDBC-15" title="resultset-seq uses structmap which has been deprecated"><del>JDBC-15</del></a> so I'm closing this out as a duplicate for now.</p> <p>Essentially duplicated with more focus in <a href="http://dev.clojure.org/jira/browse/JDBC-15" title="resultset-seq uses structmap which has been deprecated"><del>JDBC-15</del></a> and <a href="http://dev.clojure.org/jira/browse/JDBC-20" title="Create new clean API that accepts connection etc (post 0.1.0)"><del>JDBC-20</del></a>.</p> Global Rank