HeartSaVioR commented on a change in pull request #27764: [SPARK-30860][CORE]
Use FileSystem.mkdirs to avoid umask at rolling event log folder and
appStatusFile creation
URL: https://github.com/apache/spark/pull/27764#discussion_r390690674
##########
File path:
core/src/main/scala/org/apache/spark/deploy/history/EventLogFileWriters.scala
##########
@@ -361,7 +362,9 @@ class RollingEventLogFilesWriter(
private def createAppStatusFile(inProgress: Boolean): Unit = {
val appStatusPath = getAppStatusFilePath(logDirForAppPath, appId,
appAttemptId, inProgress)
- val outputStream = fileSystem.create(appStatusPath)
+ // SPARK-30860: use the class method to avoid the umask causing permission
issues
+ val outputStream = FileSystem.create(fileSystem, appStatusPath,
+ EventLogFileWriter.LOG_FILE_PERMISSIONS)
Review comment:
Yes please go ahead and fix it altogether.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
