From: | David Steele <david(at)pgmasters(dot)net> |
---|---|
To: | Stephen Frost <sfrost(at)snowman(dot)net> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Adam Brightwell <adam(dot)brightwell(at)crunchydata(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, "Tsunakawa, Takayuki" <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Configurable file mode mask |
Date: | 2018-04-07 17:09:11 |
Message-ID: | 31578b3e-6aa1-0b78-a86a-bd46b3933b4a@pgmasters.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Stephen,
On 4/6/18 10:22 PM, Stephen Frost wrote:
>
> * David Steele (david(at)pgmasters(dot)net) wrote:
>> On 4/6/18 6:04 PM, David Steele wrote:
>>> On 4/6/18 3:02 PM, Stephen Frost wrote:
>>>>
>>>> - Further discussion in the commit messages
>>>
>>> Agreed, these need some more work. I'm happy to do that but I'll need a
>>> bit more time. Have a look at the new patches and I'll work on some
>>> better messages.
>>
>> I'm sure you'll want to reword some things, but I think these commit
>> messages capture the essential changes for each patch.
>
> Thanks much. I've taken (most) of these, adjusting a few bits here and
> there.
>
> I've been back over the patch again, mostly improving the commit
> messages, comments, and docs. I also looked over the code and tests
> again and they're looking pretty good to me, so I'll be looking to
> commit this tomorrow afternoon or so US/Eastern.
OK, one last review. I did't make any code changes, but I improved some
comments, added documentation and fixed a test.
01) Refactor dir/file permissions
* Small comment update, replace "such cases-" with "such cases --".
02) Allow group access on PGDATA
* Add data_directory_mode guc to "Preset Options" documentation.
* Add a section to the documentation that discusses changing the
permissions of an existing cluster.
* Improved comment on checkControlFile().
* Comment wordsmithing for SetDataDirectoryCreatePerm() and
RetrieveDataDirCreatePerm().
* Fixed ordering of -g option in initdb documentation.
* Fixed a new test that was "broken" by 032429701e477. It was broken
before but the rmtrees added in 032429701e477 exposed the problem.
Attached are the v19 patches.
Thanks!
--
-David
david(at)pgmasters(dot)net
Attachment | Content-Type | Size |
---|---|---|
group-access-v19-01-file-perm.patch | text/plain | 43.1 KB |
group-access-v19-02-group.patch | text/plain | 49.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2018-04-07 17:19:19 | Re: pgsql: New files for MERGE |
Previous Message | Tom Lane | 2018-04-07 17:06:48 | Re: Removing useless DISTINCT clauses |