-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Track metabase.nqa module dependencies #44464
Conversation
[metabase.driver.util :as driver.u] | ||
[metabase.lib.native :as lib.native] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We still depend on both these modules, but metabase.lib
is the higher-level one.
|
This reverts commit 445230b.
(let [driver (driver.u/database->driver (:database query))] | ||
(let [driver (lib.native/engine query)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It turns out we can't actually make this change (yet?) as our test queries don't have the required :lib/type
and :stages
values used by the validation for this helper.
Description
The first stage of recovery is acceptance.
Here we declare all the module dependencies of
metabase.native-query-analyzer
.