public inbox for [email protected]  
help / color / mirror / Atom feed
From: Fabien COELHO <[email protected]>
To: Peter Eisentraut <[email protected]>
Cc: PostgreSQL Developers <[email protected]>
Subject: Re: psql - add SHOW_ALL_RESULTS option
Date: Tue, 6 Apr 2021 23:29:01 +0200 (CEST)
Message-ID: <alpine.DEB.2.22.394.2104062323150.2555247@pseudo> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<20200201120136.fnlbfcrybgzk75pc@development>
	<alpine.DEB.2.21.2002020906030.20752@pseudo>
	<alpine.DEB.2.22.394.2006061622300.32228@pseudo>
	<CA+hUKGJo1PYkW-oxzADaXDKoEUpoHonnSq4=1cUSNyOxc53Nmg@mail.gmail.com>
	<alpine.DEB.2.22.394.2007200747040.2368970@pseudo>
	<[email protected]>
	<[email protected]>
	<alpine.DEB.2.22.394.2103141046590.164573@pseudo>
	<[email protected]>


Hello Peter,

>> My 0.02€: I tested this updated version and do not have any comment on this 
>> version. From my point of view it could be committed. I would not bother to 
>> separate the test style ajustments.
>
> Committed.  The last thing I fixed was the diff in the copy2.out regression 
> test.  The order of the notices with respect to the error messages was wrong. 
> I fixed that by switching back to the regular notice processor during COPY 
> handling.
>
> Btw., not sure if that was mentioned before, but a cool use of this is to 
> \watch multiple queries at once, like
>
> select current_date \; select current_time \watch

Indeed, that was one of the things I tested on the patch. I'm wondering 
whether the documentation should point this out explicitely.

Anyway Thanks for the push!

-- 
Fabien.

view thread (104+ 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]
  Subject: Re: psql - add SHOW_ALL_RESULTS option
  In-Reply-To: <alpine.DEB.2.22.394.2104062323150.2555247@pseudo>

* 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