[spark] branch master updated: [SPARK-26928][CORE][FOLLOWUP] Fix JVMCPUSource file name and minor updates to doc

[sr...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

srowen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 4b2b3da  [SPARK-26928][CORE][FOLLOWUP] Fix JVMCPUSource file name and minor updates to doc
4b2b3da is described below

commit 4b2b3da766d90f76729dd4a684266698999161f2
Author: Luca Canali <lu...@cern.ch>
AuthorDate: Mon Mar 25 15:35:24 2019 -0500

    [SPARK-26928][CORE][FOLLOWUP] Fix JVMCPUSource file name and minor updates to doc
    
    ## What changes were proposed in this pull request?
    
    This applies some minor updates/cleaning following up SPARK-26928, notably renaming JVMCPU.scala to JVMCPUSource.scala.
    
    ## How was this patch tested?
    
    Manually tested
    
    Closes #24201 from LucaCanali/fixupSPARK-26928.
    
    Authored-by: Luca Canali <lu...@cern.ch>
    Signed-off-by: Sean Owen <se...@databricks.com>
---
 core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala    | 3 ---
 .../apache/spark/metrics/source/{JVMCPU.scala => JVMCPUSource.scala}  | 2 +-
 docs/monitoring.md                                                    | 4 ++--
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala b/core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala
index e852376..669ce63 100644
--- a/core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala
+++ b/core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala
@@ -17,12 +17,9 @@
 
 package org.apache.spark.executor
 
-import java.lang.management.ManagementFactory
 import java.util.concurrent.ThreadPoolExecutor
-import javax.management.{MBeanServer, ObjectName}
 
 import scala.collection.JavaConverters._
-import scala.util.control.NonFatal
 
 import com.codahale.metrics.{Gauge, MetricRegistry}
 import org.apache.hadoop.fs.FileSystem
diff --git a/core/src/main/scala/org/apache/spark/metrics/source/JVMCPU.scala b/core/src/main/scala/org/apache/spark/metrics/source/JVMCPUSource.scala
similarity index 97%
rename from core/src/main/scala/org/apache/spark/metrics/source/JVMCPU.scala
rename to core/src/main/scala/org/apache/spark/metrics/source/JVMCPUSource.scala
index 6ea86b8..11a3acf 100644
--- a/core/src/main/scala/org/apache/spark/metrics/source/JVMCPU.scala
+++ b/core/src/main/scala/org/apache/spark/metrics/source/JVMCPUSource.scala
@@ -19,7 +19,7 @@ package org.apache.spark.metrics.source
 
 import java.lang.management.ManagementFactory
 
-import com.codahale.metrics.{Counter, Gauge, MetricRegistry}
+import com.codahale.metrics.{Gauge, MetricRegistry}
 import javax.management.{MBeanServer, ObjectName}
 import scala.util.control.NonFatal
 
diff --git a/docs/monitoring.md b/docs/monitoring.md
index 3b187b6..bf77b4a 100644
--- a/docs/monitoring.md
+++ b/docs/monitoring.md
@@ -948,7 +948,7 @@ This is the component with the largest amount of instrumented metrics
   - LongAccumulatorSource
 
 - namespace=spark.streaming
-  - **note** applies to Spark Structured Streaming only. Conditional to a configuration
+  - **note:** This applies to Spark Structured Streaming only. Conditional to a configuration
   parameter: `spark.sql.streaming.metricsEnabled=true` (default is false) 
   - eventTime-watermark
   - inputRate-total
@@ -1076,7 +1076,7 @@ Note: applies when running in Spark standalone as worker
 - coresFree
 - memFree_MB
 
-## Component instance = shuffleService
+### Component instance = shuffleService
 Note: applies to the shuffle service
 
 - blockTransferRateBytes (meter)


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