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 |
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 |