You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/02/22 08:40:08 UTC

[GitHub] SongYadong opened a new pull request #23869: [SPARK-26967][CORE] Put MetricsSystem instance names together for clearer management

SongYadong opened a new pull request #23869: [SPARK-26967][CORE] Put MetricsSystem instance names together for clearer management
URL: https://github.com/apache/spark/pull/23869
 
 
   ## What changes were proposed in this pull request?
   
   `MetricsSystem` instance creations have a scattered distribution in the project code. So do their names. It may cause some inconvenience for browsing and management. 
   This PR tries to put them together. In this way, we can have a uniform location for adding or removing them, and have a overall view of `MetircsSystem `instances in current project.
   It's also helpful for maintaining user documents by avoiding missing something.
   
   ## How was this patch tested?
   
   Existing unit tests.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org