> On 7 Jul 2023, at 01:05, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Jacob Champion <jchampion@timescale.com> writes:
>> My two cents: backpatching an exception for .DS_Store seems good, but
>> creating an exclusion for all hidden files feels like a surprising
>> change for stable branches.
>
> On the whole, I think I'd vote for blocking .DS_Store only, even
> in HEAD. (IIRC, I thought differently to start with, but today
> I'm feeling conservative about it.) If there are other special
> file names on other platforms, we could add some more targeted
> exceptions; but dropping all hidden files seems more likely to
> break things than be helpful.
I think the case for skipping all hidden files is that it would offer more
consistency with other serverside filesystem reads are performed. After
.DS_Store I would think that editor swapfiles would be other likely culprit of
hidden-but-not-belonging files.
--
Daniel Gustafsson