[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-13 Thread kiszk
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/22397 Yes, time unit looks confusing, but not a problem. In general, due to APIs, elapsed time is `ms`, and CPU time is `ns`. --- - To

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-13 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/22397 Merged to master/2.4 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22397 **[Test build #4340 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4340/testReport)** for PR 22397 at commit

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22397 **[Test build #4340 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4340/testReport)** for PR 22397 at commit

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-12 Thread LucaCanali
Github user LucaCanali commented on the issue: https://github.com/apache/spark/pull/22397 Thanks @srowen for reviewing this. The metrics are commented in the source code of TaskMetrics class, I took most of the descriptions from there, adding some additional explanations where

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-12 Thread kiszk
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/22397 LGTM cc @jiangxb1987 @srowen --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22397 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22397 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22397: [SPARK-25170][DOC] Add list and short description of Spa...

2018-09-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/22397 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional