On 08/24/2012 10:10 AM, Tom Lane wrote:
> Magnus Hagander <magnus@hagander.net> writes:
>> On Fri, Aug 24, 2012 at 1:06 AM, Andrew Dunstan <andrew@dunslane.net> wrote:
>>> TBH I'd rather stick with the less invasive approach of the original patch
>>> at this stage, and see about refactoring for 9.3.
>> +1.
>> While I haven't looked at the code specifically, these areas can be
>> quite fragile and very environment-dependent. I'd rather not upset it
>> this close to release - especially not after RC wrap.
> Fair enough. Will one of you deal with the patch as-is, then?
>
>
I had a brief talk with Magnus the other day, and I have just spent more
time looking over this. This is a fairly narrow failure case, with a not
so narrow proposed solution. Making pg_ctl re-exec itself whenever we
see that we're running as an admin user is a very broad brush approach,
since the problem is restricted to cases where we have a config-only
data directory. I'm particularly concerned about the possible effect
that might have on pg_ctl when it's running as a service controller, and
I'm not prepared to commit anything like the current patch without a
great deal more testing. A temporary bandaid might be to do the
detection of admin privileges and go back to doing what we did there
before we got adjust_data_dir() for that case. That at least should work
no worse than what we have now.
cheers
andrew