search

seancorfield / next-jdbc

A modern low-level Clojure wrapper for JDBC-based access to databases.
https://cljdoc.org/d/com.github.seancorfield/next.jdbc/
Eclipse Public License 1.0
755 stars 90 forks source link

:column-fn in plan operations expects string #227

Closed seancorfield closed 1 year ago

seancorfield commented 1 year ago

Describe the bug The fix for #221 replaces name with (:column-fn opts) but the latter is specified to operate on strings, not keywords,

To Reproduce Run a plan operation with :column-fn str/upper-case and try select ds :id to see the failure.

Expected behavior If :column-fn is present, it should compose with name and work with keywords.