You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@hive.apache.org by GitBox <gi...@apache.org> on 2022/05/05 10:05:51 UTC
[GitHub] [hive] skysiders opened a new pull request, #3267: HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
skysiders opened a new pull request, #3267:
URL: https://github.com/apache/hive/pull/3267
HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
### What changes were proposed in this pull request?
TezSessionState, QueryResultsCache and Context use mkdirs(path, permission) to create directory with special permission. But If the umask is too restrictive, permissions may not work as expected. So we need to check if permission is set as expected.
### Does this PR introduce _any_ user-facing change?
no
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] skysiders commented on pull request #3267: HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
Posted by GitBox <gi...@apache.org>.
skysiders commented on PR #3267:
URL: https://github.com/apache/hive/pull/3267#issuecomment-1141790957
Hi @abstractdog @achennagiri ,Could you please have a look at this ?
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] skysiders commented on pull request #3267: HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
Posted by GitBox <gi...@apache.org>.
skysiders commented on PR #3267:
URL: https://github.com/apache/hive/pull/3267#issuecomment-1121956685
Hi @abstractdog , Could you please have a look at this?It same as [TEZ-4412](https://github.com/apache/tez/pull/209).
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] skysiders commented on pull request #3267: HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
Posted by GitBox <gi...@apache.org>.
skysiders commented on PR #3267:
URL: https://github.com/apache/hive/pull/3267#issuecomment-1141790958
Hi @abstractdog @achennagiri ,Could you please have a look at this ?
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] github-actions[bot] commented on pull request #3267: HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #3267:
URL: https://github.com/apache/hive/pull/3267#issuecomment-1200319102
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
Feel free to reach out on the dev@hive.apache.org list if the patch is in need of reviews.
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] github-actions[bot] closed pull request #3267: HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
Posted by GitBox <gi...@apache.org>.
github-actions[bot] closed pull request #3267: HIVE-26201 QueryResultsCache should add permission check after mkdir with special permission
URL: https://github.com/apache/hive/pull/3267
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org