Re: ExecRTCheckPerms() and many prunable partitions

Поиск
Список
Период
Сортировка
От Amit Langote
Тема Re: ExecRTCheckPerms() and many prunable partitions
Дата
Msg-id CA+HiwqEnpRFRafh5QQwq9usinvOO_Fox3oqT0x273CyanfPQgQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: ExecRTCheckPerms() and many prunable partitions  (David Rowley <dgrowleyml@gmail.com>)
Ответы Re: ExecRTCheckPerms() and many prunable partitions
Список pgsql-hackers
Дерево обсуждения
On Wed, Jun 30, 2021 at 23:34 David Rowley <dgrowleyml@gmail.com> wrote:
On Thu, 1 Jul 2021 at 01:34, Amit Langote <amitlangote09@gmail.com> wrote:
> For now, I have implemented the idea 2 as the attached patch.

I only just had a fleeting glance at the patch. Aren't you
accidentally missing the 0th RTE here?

+ while ((rti = bms_next_member(checkPermRels, rti)) > 0)
  {
- RangeTblEntry *rte = (RangeTblEntry *) lfirst(l);
+ RangeTblEntry *rte = (RangeTblEntry *) list_nth(rangeTable, rti - 1);

I'd have expected >= 0 rather than > 0.

Hmm, a valid RT index cannot be 0, so that seems fine to me.  Note that RT indexes are added as-is to that bitmapset, not after subtracting 1.
--

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