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_r393799714
##########
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:
Changed the file permission to 660 and added a separate folder permission
for 770
----------------------------------------------------------------
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]
