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
Add INFO log for each table being fingerprinted #41365
Conversation
|
(do | ||
(log/infof "Fingerprinting %s fields in table %s" (count fields) (sync-util/name-for-logging table)) |
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.
Stylistically I'd prefer putting this as a _
binding at the start of the let block to avoid so many levels of nesting.
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.
Also a smaller diff 😄
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.
mmm yeah that could look nicer. i'll try it next time
@calherries Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone? |
This PR adds INFO-level logs per table being fingerprinted. These logs will help debug performance issues with fingerprinting, because you can correlate issues with specific tables being fingerprinted.
If you add
<Logger name="metabase.sync.analyze.fingerprint" level="INFO"/>
to the log4j2.xml file, you'll get these additional lines in your logs when syncing a table for the first time: