Kimahriman 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_r390677281
##########
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:
Yeah I just used the existing permissions. Not sure why they were set with
the executable bit in the first place. I can change the files to only have 660
if there's no reason not to.
----------------------------------------------------------------
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]
