[GitHub] spark pull request #23175: [SPARK-26142]followup: Move sql shuffle read metr...

2018-11-29 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/23175


---

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



[GitHub] spark pull request #23175: [SPARK-26142]followup: Move sql shuffle read metr...

2018-11-28 Thread xuanyuanking
GitHub user xuanyuanking opened a pull request:

https://github.com/apache/spark/pull/23175

[SPARK-26142]followup: Move sql shuffle read metrics relatives to 
SQLShuffleMetricsReporter

## What changes were proposed in this pull request?

Follow up for https://github.com/apache/spark/pull/23128, move sql read 
metrics relatives to `SQLShuffleMetricsReporter`, in order to put sql shuffle 
read metrics relatives closer and avoid possible problem about forgetting 
update SQLShuffleMetricsReporter while new metrics added by others.

## How was this patch tested?

Existing tests.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xuanyuanking/spark SPARK-26142-follow

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/23175.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #23175


commit c042d15920f4ebd7166caafcb3696683978d42de
Author: Yuanjian Li 
Date:   2018-11-29T02:47:33Z

Move all sql shuffle read metrics stuff to SQLShuffleMetricsReporter




---

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