Re: WSL (windows subsystem on linux) users will need to turn fsync off as of 11.2

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: James Sewell <james(dot)sewell(at)jirotech(dot)com>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Ravi Krishna <srkrishna(at)yahoo(dot)com>, "pgsql-generallists(dot)postgresql(dot)org" <pgsql-general(at)lists(dot)postgresql(dot)org>, Bruce Klein <brucek(at)gmail(dot)com>
Subject: Re: WSL (windows subsystem on linux) users will need to turn fsync off as of 11.2
Date: 2019-02-22 21:46:32
Message-ID: CA+hUKGLbP_oj61juHTGdE2Y+wKpei8Cyo9j+arj=wzahmQRhUQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general

On Tue, Feb 19, 2019 at 5:31 PM Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> On Tue, Feb 19, 2019 at 5:16 PM James Sewell <james(dot)sewell(at)jirotech(dot)com> wrote:
> >> Here's a starter patch that shows one of the approaches discussed. It
> >> gets WSL users to a better place than they were before, by suppressing
> >> further warnings after the first one.
> >
> > This wasn't quite right, updated to check erro for ENOSYS (not rc)
> >
> > This compiles and stops the panic on WSL (with a single warning).
> >
> > I haven't tested if a version compiled on Linux will behave the same way - but based on the error messages in the top post it looks like the behavior is the same.
>
> Great. Thanks for testing, and for the fix! Well that all sounds
> like good news: it corrects the behaviour from 11.2, and also improves
> on the previous behaviour which I'd have accepted as a bug if anyone
> had reported it. So the next problem is that we don't have a
> consensus on whether this is the right approach, so I don't feel like
> I can commit it yet. Does any want to make another concrete proposal?

Ok, here's the version I'm planning to push soon if there are no objections.
Re-adding Bruce to the thread, as I just noticed the CC list got
pruned at some point in this thread.

--
Thomas Munro
https://enterprisedb.com

Attachment Content-Type Size
0001-Tolerate-ENOSYS-failure-from-sync_file_range-v3.patch application/octet-stream 2.4 KB

In response to

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Stephen Frost 2019-02-22 23:09:53 Re: streaming replication authentication
Previous Message Tiffany Thang 2019-02-22 21:00:43 Re: Copy entire schema A to a different schema B