From: | John Klann <jk7255(at)gmail(dot)com> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> |
Cc: | PostgreSQL mailing lists <pgsql-bugs(at)lists(dot)postgresql(dot)org> |
Subject: | Re: BUG #15636: PostgreSQL 11.1 pg_basebackup backup to a CIFS destination throws fsync error at end of backup |
Date: | 2019-02-15 04:27:11 |
Message-ID: | CAHyX5+W2OXAg=qEnObA3g4zU3SkwxTztSfmgmivWnWQWcsc_tA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
Happy to test it, I will need the instructions if you can. It's for Rhel
7.5 Maipo latest. (Linux 3.10.0-862.14.4.el7.x86_64 #1 SMP Fri Sep 21
09:07:21 UTC 2018
x86_64 x86_64 x86_64 GNU/Linux)
Thanks.
On Thu, Feb 14, 2019, 23:11 Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com
wrote:
> On Fri, Feb 15, 2019 at 11:53 AM John Klann <jk7255(at)gmail(dot)com> wrote:
> >>> >> Hmm, it looks like your system gives EINVAL when you try to fsync a
> >>> >> directory. Perhaps we should teach fsync__fname() about that here:
> >>> >>
> >>> >> /*
> >>> >> * Some OSes don't allow us to fsync directories at all, so
> we
> >>> >> can ignore
> >>> >> * those errors. Anything else needs to be reported.
> >>> >> */
> >>> >> if (returncode != 0 && !(isdir && errno == EBADF))
> >>> >> {
> >>> >> fprintf(stderr, _("%s: could not fsync file \"%s\":
> %s\n"),
> >>> >> progname, fname, strerror(errno));
> >>> >> (void) close(fd);
> >>> >> return -1;
> >>> >> }
>
> Here is a patch like that. Any chance you could test it? Happy to
> send instructions on how to do that if necessary. Does anyone think
> it'd be a bad idea to do this, and possibly even back-patch it? We
> have tolerated EBADF since 7d7db18a (which OS is that for?)
>
> --
> Thomas Munro
> http://www.enterprisedb.com
>
From | Date | Subject | |
---|---|---|---|
Next Message | Thomas Munro | 2019-02-15 04:46:42 | Re: BUG #15636: PostgreSQL 11.1 pg_basebackup backup to a CIFS destination throws fsync error at end of backup |
Previous Message | Thomas Munro | 2019-02-15 04:10:42 | Re: BUG #15636: PostgreSQL 11.1 pg_basebackup backup to a CIFS destination throws fsync error at end of backup |