From: | Oskari Saarenmaa <os(at)ohmu(dot)fi> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pg_basebackup fails with long tablespace paths |
Date: | 2014-12-23 08:33:44 |
Message-ID: | 549928E8.5010602@ohmu.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
23.12.2014, 05:00, Amit Kapila kirjoitti:
> On Tue, Dec 23, 2014 at 4:10 AM, Oskari Saarenmaa wrote:
>> 08.11.2014, 04:03, Peter Eisentraut kirjoitti:
>> > It errors when the file
>> > name is too long and adds tests for that. This could be applied to 9.5
>> > and backpatched, if we so choose. It might become obsolete if
>> > https://commitfest.postgresql.org/action/patch_view?id=1512 is accepted.
>> > If that patch doesn't get accepted, I might add my patch to a future
>> > commit fest.
>>
>> I think we should just use the UStar tar format
>> (http://en.wikipedia.org/wiki/Tar_%28computing%29#UStar_format) and
>> allow long file names; all actively used tar implementations should be
>> able to handle them. I'll try to write a patch for that soonish.
>>
>
> I think even using UStar format won't make it work for Windows where
> the standard utilities are not able to understand the symlinks in tar.
> There is already a patch [1] in this CF which will handle both cases, so
> I am
> not sure if it is very good idea to go with a new tar format to handle this
> issue.
>
> [1] : https://commitfest.postgresql.org/action/patch_view?id=1512
That patch makes sense for 9.5, but I don't think it's going to be
backpatched to previous releases? I think we should also apply Peter's
patch to master and backbranches to avoid creating invalid tar files
anywhere. And optionally implement and backpatch long filename support
in tar even if 9.5 no longer creates tar files with long names.
/ Oskari
From | Date | Subject | |
---|---|---|---|
Next Message | Oskari Saarenmaa | 2014-12-23 08:54:39 | Re: REINDEX CONCURRENTLY 2.0 |
Previous Message | Noah Misch | 2014-12-23 08:19:12 | Re: Initdb-cs_CZ.WIN-1250 buildfarm failures |