Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds

Поиск
Список
Период
Сортировка
Искать
От
Thomas Munro
Тема
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds
Дата
Msg-id
CA+hUKGJngeN3Sm9g2QTQd3-HxnRm-0aaPzYFJ=hCmh+T9F8Sjw@mail.gmail.com
Ответ на
Список
Дерево обсуждения
BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds PG Bug reporting form <noreply@postgresql.org>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Alexander Lakhin <exclusion@gmail.com>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Etsuro Fujita <etsuro.fujita@gmail.com>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Etsuro Fujita <etsuro.fujita@gmail.com>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Andres Freund <andres@anarazel.de>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Thomas Munro <thomas.munro@gmail.com>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Andres Freund <andres@anarazel.de>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Andres Freund <andres@anarazel.de>
Re: BUG #17828: postgres_fdw leaks file descriptors on error and aborts aborted transaction in lack of fds Etsuro Fujita <etsuro.fujita@gmail.com>
On Fri, Feb 9, 2024 at 11:06 AM Tom Lane  wrote:
> Andres Freund  writes:
> > I also wonder if postgres_fdw should strive to use a longer lived wait event
> > set. For efficiency, if nothing else? That'd avoid the need to create one
> > during error handling.
>
> Seems like a plausible idea.  Having to acquire resources to clean up
> an error is an anti-pattern for sure.

FWIW, WRT to the long lived WES idea, I have had patches to explore
several different ways to do that (as I already did successfully for
other areas), and I will write about that again later, but I concluded
that we needed to fix this Windows nonsense first:
https://commitfest.postgresql.org/46/3523/  (That got withdrawn for
being too slow, but I'll resubmit it soon enough...)


В списке pgsql-bugs по дате отправления
FAQ