skysiders opened a new pull request, #209:
URL: https://github.com/apache/tez/pull/209

   TEZ-4412 ensure mkDirForAM create directory with special permissions
   
   I found TezClientUtils has method ensureStagingDirExists.It will check 
whether the path "stagingArea" exists, if it exists, check the permission, if 
not, call the function "TezCommonUtils.mkDirForAM" to create.But in method 
mkDirForAM,it use mkdir(path, permission) to create, if umask too strict such 
as 777,this directory will set with 000 permission.So we need to ensure the 
directory has right permission.
   
   In this patch, I compared the permissions of the files with the expected 
permissions. If they are inconsistent, then log this and use setPermission to 
grant directory permissions.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to