Re: Support custom socket directory in pg_upgrade

From: Hironobu SUZUKI <hironobu(at)interdb(dot)jp>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Cc: Daniel Gustafsson <daniel(at)yesql(dot)se>
Subject: Re: Support custom socket directory in pg_upgrade
Date: 2018-10-25 13:56:04
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers


I reviewed `pg_upgrade_sockdir-v2.patch`.

I checked `-s` option on OSX. I confirmed that all tools, which are
internally invoked such as pg_dumpall and pg_restore, used the specified
socket and pg_upgrade worked as expected.

I think this patch is fine.

Best regards,

On 2018/10/09 21:26, Daniel Gustafsson wrote:
>> On 9 Oct 2018, at 16:22, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Daniel Gustafsson <daniel(at)yesql(dot)se> writes:
>>> Having hit the maximum socketdir length error a number of times in pg_upgrade,
>>> especially when running tests in a deep directory hierarchy, I figured it was
>>> time to see if anyone else has had the same problem? The attached patch is
>>> what I run with locally to avoid the issue, it adds a --socketdir=PATH option
>>> to pg_upgrade which overrides the default use of CWD. Is that something that
>>> could be considered?
>> I think you could simplify matters if you installed the CWD default value
>> during option processing.
> The attached v2 tries to make the socketdir more like the other configurable
> directories in pg_upgrade (adding an envvar for it etc). Is that more in line
> with what you were suggesting? make -C src/bin/pg_upgrade check passes with
> this, both unmodified and with a -s in the test script to override it. Also
> fixed incorrect syntax in the docs part from v1.
> cheers ./daniel

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2018-10-25 13:56:18 Re: PostgreSQL vs SQL/XML Standards
Previous Message Chapman Flack 2018-10-25 13:25:31 Re: Question about xmloption and pg_restore