Skip to content
Snippets Groups Projects
Commit 69b67c49 authored by Cam Saül's avatar Cam Saül
Browse files

Fix for Mongo [ci drivers] :wrench:

parent 8d5e42fe
No related branches found
No related tags found
No related merge requests found
......@@ -166,7 +166,7 @@
since at this time we can't resolve those normally (#1786) fall back to using the metadata for the first column (e.g., `count`).
This is definitely a HACK, but in most cases this should be correct (or at least better than the generic info) for the important things like type information."
[fields k]
(when-let [[_ field-name-without-suffix] (re-matches #"(.*)_\d+$" (name k))]
(when-let [[_ field-name-without-suffix] (re-matches #"^(.*)_\d+$" (name k))]
(some (fn [{field-name :field-name, :as field}]
(when (= (name field-name) field-name-without-suffix)
(merge (generic-info-for-missing-key k)
......
......@@ -149,7 +149,8 @@
(ql/aggregation (ql/avg $price) (ql/count) (ql/sum $price))))))
;; make sure that multiple aggregations of the same type have the correct metadata (#4003)
(expect-with-non-timeseries-dbs
;; (TODO - this isn't tested against Mongo because the Mongo driver currently combines duplicate aggregations of the same time into a single type)
(datasets/expct-with-engines (disj non-timeseries-engines :mongo)
[(aggregate-col :count)
(assoc (aggregate-col :count)
:display_name "count_2"
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment