zephyring commented on PR #25104: URL: https://github.com/apache/superset/pull/25104#issuecomment-1696396832
> @zephyring I assume this is a reaction to this recent PR: #25080. Did this cause issues in your env? I assumed the change would be very low impact, as we have a fair amount of StatsD metrics throughout the code base. Just curious so I understand what adverse implications this may have had.. 1. In preset we have lots of superset instances and each instance will start emitting a stats log every minutes. This is gonna be too noisy for us. 2. I think providing a config here will make it explicit that we're doing this stats log in report schedule so if other parties can also leverage if needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
