-
Notifications
You must be signed in to change notification settings - Fork 9.2k
YARN-11117 check permission for LeveldbRMStateStore #4243
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
Conversation
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
Hi @hemanthboyina ,Could you take a look at this. Thanks for your review. |
hemanthboyina
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi @skysiders can you extend an UT for this
|
We're closing this stale PR because it has been open for 100 days with no activity. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. |
YARN-11117 check permission for LeveldbRMStateStore
Description of PR
LeveldbRMStateStore use fs.mkdirs(root,new FsPermission((short)0700)); to create root directory with permission 700.BUT if umask is too strict such as 0777, this directory will have wrong permission with 000. So it should check if umask can affect permission, if true , use setPermission to fix it
How was this patch tested?
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?