From 4dcb015dc92db4b56847b482cef920f8545b996f Mon Sep 17 00:00:00 2001
From: github-automation-metabase
<166700802+github-automation-metabase@users.noreply.github.com>
Date: Fri, 20 Dec 2024 15:32:56 -0500
Subject: [PATCH] Add debug logging for chain filter query (#51481) (#51594)
* Add debug logging for chain filter query
* Comment
Co-authored-by: lbrdnk <lbrdnk@users.noreply.github.com>
---
src/metabase/models/params/chain_filter.clj | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/src/metabase/models/params/chain_filter.clj b/src/metabase/models/params/chain_filter.clj
index 46446980fc4..3733fb8b546 100644
--- a/src/metabase/models/params/chain_filter.clj
+++ b/src/metabase/models/params/chain_filter.clj
@@ -80,6 +80,7 @@
[metabase.models.params.chain-filter.dedupe-joins :as dedupe]
[metabase.models.params.field-values :as params.field-values]
[metabase.query-processor :as qp]
+ [metabase.query-processor.compile :as qp.compile]
[metabase.query-processor.middleware.permissions :as qp.perms]
[metabase.query-processor.preprocess :as qp.preprocess]
[metabase.query-processor.setup :as qp.setup]
@@ -442,6 +443,11 @@
;; 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]
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
;; 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.
--
GitLab