Re: fsync-pgdata-on-recovery tries to write to more files than previously
| От | Amit Kapila |
|---|---|
| Тема | Re: fsync-pgdata-on-recovery tries to write to more files than previously |
| Дата | |
| Msg-id | CAA4eK1Kes+yCqLbVzSRkHrVcMEaUCkuRsuA10V105KbrYXFxKQ@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: fsync-pgdata-on-recovery tries to write to more files than previously (Michael Paquier <michael.paquier@gmail.com>) |
| Ответы |
Re: fsync-pgdata-on-recovery tries to write to more files
than previously
|
| Список | pgsql-hackers |
On Fri, May 29, 2015 at 2:28 PM, Michael Paquier <michael.paquier@gmail.com> wrote:
>
> On Fri, May 29, 2015 at 5:01 PM, Amit Kapila <amit.kapila16@gmail.com> wrote:
> >
> > Test-3 - Symlinks in pg_tblspc.
> > 1. Create couple of tablespaces which creates symlinks
> > in pg_tblspc
> > 2. Crash the server
> > 3. Restart Server - It works fine.
> >
> > I am not sure Test-2 is a valid thing and we should support it or
> > not, but the current patch is sane w.r.t that form of symlinks
> > as well.
>
> It is always good to check, but is that relevant to the bug fixed? I
> mean, you need to symlink a file in PGDATA that server has no write
> permission on... For example tablespaces can be written to in test 3,
> so that would work even with 9.4.2 after crashing the server with -m
> immediate.
>
> On Fri, May 29, 2015 at 5:01 PM, Amit Kapila <amit.kapila16@gmail.com> wrote:
> >
> > Test-3 - Symlinks in pg_tblspc.
> > 1. Create couple of tablespaces which creates symlinks
> > in pg_tblspc
> > 2. Crash the server
> > 3. Restart Server - It works fine.
> >
> > I am not sure Test-2 is a valid thing and we should support it or
> > not, but the current patch is sane w.r.t that form of symlinks
> > as well.
>
> It is always good to check, but is that relevant to the bug fixed? I
> mean, you need to symlink a file in PGDATA that server has no write
> permission on... For example tablespaces can be written to in test 3,
> so that would work even with 9.4.2 after crashing the server with -m
> immediate.
I have just tried to cover the paths which has symlink related code
in the new commit (in functions SyncDataDirectory() and walkdir()),
because thats what I understood from Tom's mail. Without test-3, it
won't cover walkdir symlink test path, I didn't knew that there was any
confusion about verification of permissions problem on Windows
and I haven't looked to verify the whole patch.
В списке pgsql-hackers по дате отправления: