Re: GenBKI emits useless open;close for catalogs without rows

Поиск
Список
Период
Сортировка
От vignesh C
Тема Re: GenBKI emits useless open;close for catalogs without rows
Дата
Msg-id CALDaNm1G97dKHRPhUohYPZASfEduD6BcAN9nu6USPicR7sPDwQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: GenBKI emits useless open;close for catalogs without rows  (Peter Eisentraut <peter@eisentraut.org>)
Список pgsql-hackers
On Wed, 8 Nov 2023 at 12:50, Peter Eisentraut <peter@eisentraut.org> wrote:
>
> On 08.11.23 08:16, Peter Eisentraut wrote:
> > On 19.09.23 20:05, Heikki Linnakangas wrote:
> >> One thing caught my eye though: We currently have an "open" command
> >> after every "create". Except for bootstrap relations; creating a
> >> bootstrap relation opens it implicitly. That seems like a weird
> >> inconsistency. If we make "create" to always open the relation, we can
> >> both make it more consistent and save a few lines. That's maybe worth
> >> doing, per the attached. It removes the "open" command altogether, as
> >> it's not needed anymore.
> >
> > This seems like a good improvement to me.
> >
> > It would restrict the bootstrap language so that you can only manipulate
> > a table right after creating it, but I don't see why that wouldn't be
> > sufficient.
>
> Then again, this sort of achieves the opposite of what Matthias was
> aiming for: You are now forcing some relations to be opened even though
> we will end up closing it right away.
>
> (In any case, documentation in bki.sgml would need to be updated for
> this patch.)

I have changed the status of the patch to WOA, feel free to update the
status once Peter's documentation comments are addressed.

Regards,
Vignesh



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

Предыдущее
От: vignesh C
Дата:
Сообщение: Re: Using AF_UNIX sockets always for tests on Windows
Следующее
От: Tom Lane
Дата:
Сообщение: Re: 003_extrafiles.pl test fails on Windows with the newer Perl versions