mridulm commented on pull request #31992: URL: https://github.com/apache/spark/pull/31992#issuecomment-813543189
As I understood based on description and proposed changes, the right level of integration for this would be with metrics subsystem.. While we can do it via other means, it is not optimal to do so. Having said that, I dont have strong opinions on this ... will let some more involved with this comment better. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
