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

Make sure BigQuery driver declares :basic-aggregations feature [ci drivers]

parent 58b262eb
No related branches found
No related tags found
No related merge requests found
......@@ -419,9 +419,9 @@
;; people can manually specifiy "foreign key" relationships in admin and everything should work correctly.
;; Since we can't infer any "FK" relationships during sync our normal FK tests are not appropriate for BigQuery, so they're disabled for the time being.
;; TODO - either write BigQuery-speciifc tests for FK functionality or add additional code to manually set up these FK relationships for FK tables
:features (constantly (when-not config/is-test?
;; during unit tests don't treat bigquery as having FK support
#{:foreign-keys}))
:features (constantly (set/union #{:basic-aggregations} (when-not config/is-test?
;; during unit tests don't treat bigquery as having FK support
#{:foreign-keys})))
:field-values-lazy-seq (u/drop-first-arg field-values-lazy-seq)
:mbql->native (u/drop-first-arg mbql->native)}))
......
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