Re: mislead comments in pg_get_statisticsobjdef_string

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: jian he <jian(dot)universality(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: mislead comments in pg_get_statisticsobjdef_string
Date: 2025-09-25 23:05:17
Message-ID: CAApHDvoLZMH6ruY9BXj31D5zx-=NHeyLVw5QiCDJp7sGjqcjxQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, 26 Sept 2025 at 01:17, jian he <jian(dot)universality(at)gmail(dot)com> wrote:
> * Internal version for use by ALTER TABLE.
> * Includes a tablespace clause in the result.
> * Returns a palloc'd C string; no pretty-printing.
> */
> char *
> pg_get_statisticsobjdef_string(Oid statextid)
> {
> return pg_get_statisticsobj_worker(statextid, false, false);
> }
>
> is the above "tablespace clause" comment wrong?

Yes

> since there is no "tablespace clause" related to CREATE STATISTICS.
>
> I guess that time, comments were just simply copied from
> function pg_get_indexdef_string.

I expect you're correct. I've pushed a fix. Thanks

David

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-09-25 23:49:11 Re: Orphan page in _bt_split
Previous Message David Rowley 2025-09-25 22:23:23 Re: Remove unused parameter on check_and_push_window_quals