Horton

Re: logical decoding : exceeded maxAllocatedDescs for .spill files

Amit Khandekar <amitdkhan.pg@gmail.com>

View thread
From: Amit Khandekar <amitdkhan.pg@gmail.com>
To: Amit Kapila <amit.kapila16@gmail.com>
Cc: Alvaro Herrera from 2ndQuadrant <alvherre@alvh.no-ip.org>, Andres Freund <andres@anarazel.de>, Juan José Santamaría Flecha <juanjo.santamaria@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Robert Haas <robertmhaas@gmail.com>, Thomas Munro <thomas.munro@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, Tomas Vondra <tomas.vondra@2ndquadrant.com>
Date: 2019-11-22T14:08:20Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. When a TAP file has non-zero exit status, retain temporary directories.

  2. Fix running out of file descriptors for spill files.

  3. Track statistics for spilling of changes from ReorderBuffer.

  4. Handle ReadFile() EOF correctly on Windows.

  5. Add logical_decoding_work_mem to limit ReorderBuffer memory usage.

  6. Generational memory allocator

  7. Support retaining data dirs on successful TAP tests

On Fri, 22 Nov 2019 at 4:26 PM, Amit Kapila <amit.kapila16@gmail.com> wrote:

> On Fri, Nov 22, 2019 at 11:00 AM Amit Khandekar <amitdkhan.pg@gmail.com>
> wrote:
> >
> > On Fri, 22 Nov 2019 at 09:08, Amit Kapila <amit.kapila16@gmail.com>
> wrote:
> > > Have you tried before that fix , if not, can you once try by
> > > temporarily reverting that fix in your environment and share the
> > > output of each step?  After you get the error due to EOF, check that
> > > you have .spill files in pg_replslot/<slot_name>/ and then again try
> > > to get changes by pg_logical_slot_get_changes().   If you want, you
> > > can use the test provided in Amit Khandekar's patch.
> >
> > On my Linux machine, I added elog() in ReorderBufferRestoreChanges(),
> > just after FileRead() returns 0. This results in error. But the thing
> is, in
> > ReorderBufferCommit(), the error is already handled using PG_CATCH :
> >
> > PG_CATCH();
> > {
> > .....
> >    AbortCurrentTransaction();
> > .......
> >    if (using_subtxn)
> >       RollbackAndReleaseCurrentSubTransaction();
> > ........
> > ........
> >    /* remove potential on-disk data, and deallocate */
> >    ReorderBufferCleanupTXN(rb, txn);
> > }
> >
> > So ReorderBufferCleanupTXN() removes all the .spill files using unlink().
> >
> > And on Windows, what should happen is : unlink() should succeed
> > because the file is opened using FILE_SHARE_DELETE. But the files
> > should still remain there because these are still open. It is just
> > marked for deletion until there is no one having opened the file. That
> > is what is my conclusion from running a sample attached program test.c
> >
>
> I think this is exactly the reason for the problem.  In my test [1],
> the error "permission denied" occurred when I second time executed
> pg_logical_slot_get_changes() which means on first execution the
> unlink would have been successful but the files are still not removed
> as they were not closed. Then on second execution, it gets an error
> "Permission denied" when it again tries to unlink files via
> ReorderBufferCleanupSerializedTXNs().
>
>
> .
> > But what you are seeing is "Permission denied" errors. Not sure why
> > unlink() is failing.
> >
>
> In your test program, if you try to unlink the file second time, you
> should see the error "Permission denied".

 I tested using the sample program and indeed I got the error 5 (access
denied) when I called unlink the second time.

>
>
> [1] -
> https://www.postgresql.org/message-id/CAA4eK1%2Bcey6i6a0zD9kk_eaDXb4RPNZqu4UwXO9LbHAgMpMBkg%40mail.gmail.com
>
>
>
> --
> With Regards,
> Amit Kapila.
> EnterpriseDB: http://www.enterprisedb.com
>
-- 
Thanks,
-Amit Khandekar
EnterpriseDB Corporation
The Postgres Database Company