Hi,
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@sss.pgh.pa.us> wrote:
>>
>> Daniel Gustafsson <daniel@yesql.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
>
>
>
>
>
>
