Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-22174][CORE]Support to automatically create the directory where the event logs go (spark.eventLog.dir) #19398

Closed
wants to merge 1 commit into from

Conversation

zuotingbing
Copy link

What changes were proposed in this pull request?

2017-09-30 09:47:44,721 ERROR org.apache.spark.SparkContext: Error initializing SparkContext. java.io.FileNotFoundException: File file:/tmp/spark-events does not exist at org.apache.hadoop.fs.RawLocalFileSystem.deprecatedGetFileStatus(RawLocalFileSystem.java:611) at org.apache.hadoop.fs.RawLocalFileSystem.getFileLinkStatusInternal(RawLocalFileSystem.java:824) at org.apache.hadoop.fs.RawLocalFileSystem.getFileStatus(RawLocalFileSystem.java:601) at org.apache.hadoop.fs.FilterFileSystem.getFileStatus(FilterFileSystem.java:421) at org.apache.spark.scheduler.EventLoggingListener.start(EventLoggingListener.scala:93) at org.apache.spark.SparkContext.<init>(SparkContext.scala:516) at org.apache.spark.SparkContext$.getOrCreate(SparkContext.scala:2258) at org.apache.spark.sql.SparkSession$Builder$$anonfun$8.apply(SparkSession.scala:846) at org.apache.spark.sql.SparkSession$Builder$$anonfun$8.apply(SparkSession.scala:838) at scala.Option.getOrElse(Option.scala:121) at org.apache.spark.sql.SparkSession$Builder.getOrCreate(SparkSession.scala:838) at org.apache.spark.examples.SparkPi$.main(SparkPi.scala:31) at org.apache.spark.examples.SparkPi.main(SparkPi.scala)

Currently, if our applications are using event logging, the directory where the event logs go (spark.eventLog.dir) should be manually created.
I suggest to create the event log directory automatically in source code, this will make spark more convenient to use.

How was this patch tested?

update existing tests

…e the event logs go (`spark.eventLog.dir`)
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@jerryshao
Copy link
Contributor

AFAIK, this is a by-design choice to manually create event log directory.

@vanzin
Copy link
Contributor

vanzin commented Sep 30, 2017

See bug update. Please close this PR.

@zuotingbing
Copy link
Author

Do you know what is the reason to choice to manually create? What about changing it to create it automatically? Thanks.

@zuotingbing
Copy link
Author

Got it, thanks all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants