point_up::skin-tone-2 I've bumped up the minor version here because of the metadata support on :select. {:select ^:distinct [:a :b] ..} will produce SELECT DISTINCT a, b .. (as an alternative to using :select-distinct. I added this as a way to support BigQuery's SELECT AS STRUCT: {:select ^{:as :struct} [:a :b] ..} ;;=> SELECT AS STRUCT a, b .. but it will handle any basic metadata there (keyword/symbol values only!). It excludes :line and :column in case you are producing this data via macros (I ran into that during testing). Please let me know if you run into any issues with other metadata keys that need to be excluded.
The NRQL dialect support is fairly minimal so far but covers the common clauses that New Relic have added to their SQL implementation and handles the somewhat unusual stropping (quoting) and auto-inlines everything.
@seancorfield the artifact version on clojars is still 2.4 https://clojars.org/com.github.seancorfield/honeysql/versions/2.4.1090
That's because I'm an idiot 🙂 Thank you!
it happens, thanks for your work on this library, I use it all the time!
OK, that should be {:mvn/version "2.5.1091"} on Clojars now! Thanks to @skynet for pointing out my mistake.
on this, my work colleague ran into the metadata issue when something was adding :end-line and :end-column with numbers as values. (`file` was also there but that had a string value)
They were using doomacs so maybe something on cider / refactor-nrepl side added the extra key-values
It isn't so much about the values being numbers as the metadata being turned into SQL. I can add those to the ignore list but any "unknown" metadata would be treated as something to format into the generated SQL...