[GitHub] drill pull request #820: DRILL-5391: CTAS: make folder and file permission c...

2017-05-01 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/drill/pull/820


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] drill pull request #820: DRILL-5391: CTAS: make folder and file permission c...

2017-04-26 Thread arina-ielchiieva
GitHub user arina-ielchiieva opened a pull request:

https://github.com/apache/drill/pull/820

DRILL-5391: CTAS: make folder and file permission configurable

1. Added new configuration option exec.persistent_table.umask which 
defaults to 002. User can modify this option on session or system level. If 
umask was set incorrectly, default umask will be used (002) and error will be 
logged.
2. Refactored `StorageStrategy` to use umask.
3. Added appropriate unit tests.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/arina-ielchiieva/drill DRILL-5391

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/drill/pull/820.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #820


commit 3ed005864cdac8d7d04d61307fe1493b38e1ea97
Author: Arina Ielchiieva 
Date:   2017-04-26T13:27:19Z

DRILL-5391: CTAS: make folder and file permission configurable




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---