From cd2cd950d02881b01cf3aee58ce885a506a52002 Mon Sep 17 00:00:00 2001
From: Alexander Solovyov <alexander@solovyov.net>
Date: Mon, 14 Oct 2024 11:15:06 +0300
Subject: [PATCH] [ParseSQL] log less warnings/errors (#48614)
Two reasons: it's very noisy, and the feature is not very user-visible, so does not make a lot of sense yet
---
src/metabase/query_analysis.clj | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/metabase/query_analysis.clj b/src/metabase/query_analysis.clj
index e62fea3268f..d09e578f012 100644
--- a/src/metabase/query_analysis.clj
+++ b/src/metabase/query_analysis.clj
@@ -144,9 +144,7 @@
success? (some? references)]
(if-not success?
- (do
- (log/errorf "Failed to analyze query for card %s" card-id)
- (t2/update! :model/QueryAnalysis analysis-id {:status "failed"}))
+ (t2/update! :model/QueryAnalysis analysis-id {:status "failed"})
(do
(t2/insert! :model/QueryField (map field->row (:fields references)))
(t2/insert! :model/QueryTable (map table->row (:tables references)))
@@ -225,7 +223,7 @@
card-id (:id card)]
(cond
(not card) (log/warnf "Card not found: %s" card-id)
- (:archived card) (log/warnf "Skipping archived card: %s" card-id)
+ (:archived card) (log/debugf "Skipping archived card: %s" card-id)
:else (do
(log/debugf "Performing query analysis for card %s" card-id)
(update-query-analysis-for-card! card)))))
--
GitLab