Re: doc: alter table references bogus table-specific planner parameters

From: Justin Pryzby <pryzby(at)telsasoft(dot)com>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: doc: alter table references bogus table-specific planner parameters
Date: 2020-01-22 01:27:47
Message-ID: 20200122012747.GJ26045@telsasoft.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jan 06, 2020 at 04:33:46AM +0000, Simon Riggs wrote:
> On Mon, 6 Jan 2020 at 04:13, Justin Pryzby <pryzby(at)telsasoft(dot)com> wrote:
> > > I agree with the sentiment of the third doc change, but your patch removes
> > > the mention of n_distinct, which isn't appropriate.
> >
> > I think it's correct to remove n_distinct there, as it's documented previously,
> > since e5550d5f. That's a per-attribute option (not storage) and can't be
> > specified there.
>
> OK, then agreed.

Attached minimal patch with just this hunk.

https://commitfest.postgresql.org/27/2417/
=> RFC

Justin

(I'm resending in a new thread since it looks like the first message was
somehow sent as a reply to an unrelated thread.)

Attachment Content-Type Size
v2-0001-doc-alter-table-references-bogus-table-specific-p.patch text/x-diff 1.3 KB

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Masahiko Sawada 2020-01-22 01:43:45 Re: [HACKERS] Block level parallel vacuum
Previous Message Justin Pryzby 2020-01-22 01:17:52 Re: error context for vacuum to include block number