Re: Pgoutput not capturing the generated columns

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: Peter Smith <smithpb2250(at)gmail(dot)com>, "Shinoda, Noriyoshi (SXD Japan FSI)" <noriyoshi(dot)shinoda(at)hpe(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, vignesh C <vignesh21(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Shubham Khanna <khannashubham1197(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Rajendra Kumar Dangwal <dangwalrajendra888(at)gmail(dot)com>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "euler(at)eulerto(dot)com" <euler(at)eulerto(dot)com>
Subject: Re: Pgoutput not capturing the generated columns
Date: 2025-01-24 04:21:46
Message-ID: 291928.1737692506@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> writes:
> On Fri, Jan 24, 2025 at 8:39 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> I think the problem is not so much the underscore as the
>> inconsistency. You've got "pub", "gen", and "cols" run together,
>> but then you feel a need to separate "type"?

> It was easy to read and to avoid getting a single word too long.
> However, I do understand your concern. so will change it to
> pubgencolstype unless you or someone prefers pubgencols?

I think I'd vote for "pubgencols". I don't see what the "_type"
suffix is supposed to convey --- there is nothing very type-y about
this.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Shlok Kyal 2025-01-24 04:22:41 Re: create subscription with (origin = none, copy_data = on)
Previous Message Amit Kapila 2025-01-24 04:15:29 Re: Pgoutput not capturing the generated columns