Skip to content
Snippets Groups Projects
Unverified Commit 4dcb015d authored by github-automation-metabase's avatar github-automation-metabase Committed by GitHub
Browse files

Add debug logging for chain filter query (#51481) (#51594)


* Add debug logging for chain filter query

* Comment

Co-authored-by: default avatarlbrdnk <lbrdnk@users.noreply.github.com>
parent 8ad2516b
No related branches found
No related tags found
No related merge requests found
...@@ -80,6 +80,7 @@ ...@@ -80,6 +80,7 @@
[metabase.models.params.chain-filter.dedupe-joins :as dedupe] [metabase.models.params.chain-filter.dedupe-joins :as dedupe]
[metabase.models.params.field-values :as params.field-values] [metabase.models.params.field-values :as params.field-values]
[metabase.query-processor :as qp] [metabase.query-processor :as qp]
[metabase.query-processor.compile :as qp.compile]
[metabase.query-processor.middleware.permissions :as qp.perms] [metabase.query-processor.middleware.permissions :as qp.perms]
[metabase.query-processor.preprocess :as qp.preprocess] [metabase.query-processor.preprocess :as qp.preprocess]
[metabase.query-processor.setup :as qp.setup] [metabase.query-processor.setup :as qp.setup]
...@@ -442,6 +443,11 @@ ...@@ -442,6 +443,11 @@
;; FIXME: this can OOM for text column if each value are too large. See #46411 ;; FIXME: this can OOM for text column if each value are too large. See #46411
;; Consider using the [[field-values/distinct-text-field-rff] rff] ;; Consider using the [[field-values/distinct-text-field-rff] rff]
values (qp/process-query mbql-query (constantly conj))] values (qp/process-query mbql-query (constantly conj))]
(try ; Feature issue #46888: log chain filter query.
(log/debugf "Chain filter native query: `%s`."
(:query (qp.compile/compile mbql-query)))
(catch Throwable _
(log/error "Chain filter log failed!")))
{:values values {:values values
;; It's unlikely that we don't have a query-limit, but better safe than sorry and default it true ;; It's unlikely that we don't have a query-limit, but better safe than sorry and default it true
;; so that calling chain-filter-search on the same field will search from DB. ;; so that calling chain-filter-search on the same field will search from DB.
......
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