public inbox for [email protected]  

help / color / mirror / Atom feed

From: Gilles Darold <[email protected]>
To: Nathan Bossart <[email protected]>
To: Gilles Darold <[email protected]>
Cc: Andrew Dunstan <[email protected]>
Cc: Justin Pryzby <[email protected]>
Cc: [email protected]
Cc: Robert Haas <[email protected]>
Subject: Re: fix and document CLUSTER privileges
Date: Wed, 4 Jan 2023 23:27:05 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <20230104181856.GC296349@nathanxps13>
References: <20221207223924.GA4182184@nathanxps13>
	<[email protected]>
	<20221208041313.GA216874@nathanxps13>
	<[email protected]>
	<20221208181324.GA4385@nathanxps13>
	<CA+TgmoaWhAjTP3a9xjA2P=xkF-RV9UOYT11JVWgQq5A8AB9ZrA@mail.gmail.com>
	<20221214173435.GA690225@nathanxps13>
	<20221216045700.GA804311@nathanxps13>
	<[email protected]>
	<20230104181856.GC296349@nathanxps13>

Le 04/01/2023 à 19:18, Nathan Bossart a écrit :
> On Wed, Jan 04, 2023 at 02:25:13PM +0100, Gilles Darold wrote:
>> This is the current behavior of the CLUSTER command and current patch adds a
>> sentence about the silent behavior in the documentation. This is good but I
>> just want to ask if we could want to fix this behavior too or just keep
>> things like that with the lack of noise.
> I've proposed something like what you are describing in another thread [0].
> I intended to simply fix and document the current behavior in this thread
> and to take up any new changes in the other one.
>
> [0] https://commitfest.postgresql.org/41/4070/


Got it, this is patch add_cluster_skip_messages.patch . IMHO this patch 
should be part of this commitfest as it is directly based on this one. 
You could create a second patch here that adds the warning message so 
that committers can decide here if it should be applied.


-- 
Gilles Darold

view thread (3+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: fix and document CLUSTER privileges
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox