Skip to content

Conversation

@mistercrunch
Copy link
Member

fixes #3660

@coveralls
Copy link

coveralls commented Oct 13, 2017

Coverage Status

Coverage remained the same at 70.113% when pulling 887a497 on mistercrunch:stats_debug into f871634 on apache:master.

@timifasubaa
Copy link
Contributor

lgtm

@rumbin
Copy link
Contributor

rumbin commented Oct 19, 2017

@mistercrunch Are you sure that this fixes #3660? I included your changes in our installation and restarted superset, and these log messages still are flooding the console log. I don't assume that I was supposed to build anything here, as this is only a py file, right?

@xrmx
Copy link
Contributor

xrmx commented Oct 19, 2017

@rumbin you have default log level set to debug, you have to raise it to something less spammy.

@rumbin
Copy link
Contributor

rumbin commented Oct 19, 2017

Well this is embarassing.
I didn't notice that the default LOG_LEVEL is set to DEBUG.

Thanks anyway, @xrmx...

@mistercrunch mistercrunch merged commit e97dc9d into apache:master Oct 25, 2017
michellethomas pushed a commit to michellethomas/panoramix that referenced this pull request May 24, 2018
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.20.5 First shipped in 0.20.5 labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.20.5 First shipped in 0.20.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[stats_logger] entries flooding the console log

5 participants