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

    https://github.com/apache/spark/pull/1067#discussion_r13703371
  
    --- Diff: core/src/main/scala/org/apache/spark/metrics/sink/CsvSink.scala 
---
    @@ -53,11 +53,14 @@ private[spark] class CsvSink(val property: Properties, 
val registry: MetricRegis
         case None => CSV_DEFAULT_DIR
       }
     
    +  val file= new File(pollDir + conf.get("spark.app.uniqueName"))
    --- End diff --
    
    Forgot to mention that several sinks like CsvSink will be started in master 
and worker for standalone mode if configured, in this situation you may not get 
unique app name or SparkEnv (something created only when app started). So maybe 
you should consider this situation when there's no app name available.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to