Re: SupportRequestRows support function for generate_series_timestamptz

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: jian he <jian(dot)universality(at)gmail(dot)com>
Cc: PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: SupportRequestRows support function for generate_series_timestamptz
Date: 2024-07-08 05:16:00
Message-ID: CAApHDvrOTw9rT3WKZHHO+2hzjoW-0skpxRArF6=vgSVrVOTD-g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, 8 Jul 2024 at 16:43, jian he <jian(dot)universality(at)gmail(dot)com> wrote:
> { oid => '2031',
> proname => 'timestamp_mi', prorettype => 'interval',
> proargtypes => 'timestamp timestamp', prosrc => 'timestamp_mi' },
> { oid => '1188',
> proname => 'timestamptz_mi', prorettype => 'interval',
> proargtypes => 'timestamptz timestamptz', prosrc => 'timestamp_mi' },

I'm not quite sure what you mean that needs to be adjusted with this.

> so this also apply to
>
> { oid => '938', descr => 'non-persistent series generator',
> proname => 'generate_series', prorows => '1000', proretset => 't',
> prorettype => 'timestamp', proargtypes => 'timestamp timestamp interval',
> prosrc => 'generate_series_timestamp' },
>
> If so, then we need to update src/include/catalog/pg_proc.dat also?

Oh, yeah. I missed setting the prosupport function for that one. Thanks.

I'm not sure when I realised there were 3 of these functions, but it
seems I didn't when I adjusted pg_proc.dat.

Updated patch attached.

David

Attachment Content-Type Size
v4-0001-Add-support-function-for-generate_series-for-time.patch application/octet-stream 15.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Smith 2024-07-08 05:22:54 Re: Pgoutput not capturing the generated columns
Previous Message jian he 2024-07-08 04:43:45 Re: SupportRequestRows support function for generate_series_timestamptz