Re: [HACKERS] [PATCH] pageinspect function to decode infomasks

Поиск
Список
Период
Сортировка
От Amit Kapila
Тема Re: [HACKERS] [PATCH] pageinspect function to decode infomasks
Дата
Msg-id CAA4eK1+GBB4L_WeKcpBEXUcnWC1ohpzKx1xZOzLZAPjRZ+5vKw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [HACKERS] [PATCH] pageinspect function to decode infomasks  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: [HACKERS] [PATCH] pageinspect function to decode infomasks  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers
On Sat, Sep 14, 2019 at 3:00 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Sat, Sep 14, 2019 at 03:03:57PM +0900, Michael Paquier wrote:
> > On Sat, Sep 14, 2019 at 11:18:37AM +0530, Amit Kapila wrote:
> >> Won't 'Lateral' clause be helpful here as the patch contains it in one
> >> of its tests?
> >
> > Ah true, I forgot that.
>
> If we are redesigning the interface, here are two extra thoughts which
> may be worth considering:
> 1) If the function returns multiple columns, could it make sense to
> separate infomask and infomask2?  This would then give 3 columns:
> - The raw flags for infomask.
> - The three combined flags for infomask.
> - The flags for infomask2.
> 2) Could it make sense to have a separate function for infomask2?
>

I don't see much use of separating information for infomask and infomask2.

-- 
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com



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

Предыдущее
От: nilsocket
Дата:
Сообщение: Re:
Следующее
От: Tattsu Yama
Дата:
Сообщение: Re: [HACKERS] CLUSTER command progress monitor