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

[ParseSQL] log less warnings/errors (#48614) (#48660)


Two reasons: it's very noisy, and the feature is not very user-visible, so does not make a lot of sense yet

Co-authored-by: default avatarAlexander Solovyov <alexander@solovyov.net>
parent 85df72e0
Branches
Tags
No related merge requests found
......@@ -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)))))
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment