Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22218#discussion_r214484633
  
    --- Diff: 
core/src/main/scala/org/apache/spark/executor/ExecutorSource.scala ---
    @@ -73,6 +75,29 @@ class ExecutorSource(threadPool: ThreadPoolExecutor, 
executorId: String) extends
         registerFileSystemStat(scheme, "write_ops", _.getWriteOps(), 0)
       }
     
    +  /** Dropwizard metrics gauge measuring the executor's process CPU time.
    --- End diff --
    
    Nit: the comments should begin on the next line. But this is scaladoc 
syntax, and inside a code block, normally we just use `//` block comments.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to