pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Anastasia Lubennikova <a.lubennikova@postgrespro.ru>
Cc: Greg Nancarrow <gregn4422@gmail.com>
Cc: Peter Smith <smithpb2250@gmail.com>
Cc: Smith, Peter <peters@fast.au.fujitsu.com>
Cc: David Steele <david@pgmasters.net>
Cc: Tsunakawa, Takayuki <tsunakawa.takay@fujitsu.com>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Haribabu Kommi <kommi.haribabu@gmail.com>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Laurenz Albe <laurenz.albe@cybertec.at>
Cc: Dave Cramer <pg@fastcrypt.com>
Cc: Jing Wang <jingwangian@gmail.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Cc: Elvis Pranskevichus <elprans@gmail.com>
Subject: Re: Libpq support to connect to standby server as priority
Date: Tue, 24 Nov 2020 16:49:19 -0500
Message-ID: <311787.1606254559@sss.pgh.pa.us> (raw)
In-Reply-To: <91749879-e086-02db-a4dd-44d49f0332b2@postgrespro.ru>
References: <20200228160535.GA32597@alvherre.pgsql>
	<abe22229-3f22-13d6-ae4d-afd87774e549@pgmasters.net>
	<CAJcOf-dMEKXxMVgRB5N1J3RXbW91uOyAJ0jxVA5edX7Ky3995Q@mail.gmail.com>
	<4769B138-389C-436A-A025-3CA3468ECD16@yesql.se>
	<CAJcOf-fYvLQEOG9ThE=AxpuA4Retnvjh7Q4MrBfyy43yDDK9kQ@mail.gmail.com>
	<87EDE4DB-B82B-43EB-A349-EED4643F3C1D@yesql.se>
	<CAJcOf-cEMJMt5_b466+kwP4szosnnqTQft1M5=f8aAQzSV-sWA@mail.gmail.com>
	<9a795e59e0834b13bf8054107a491e57@G06AUEXCASMHB04.g06.fujitsu.local>
	<CAJcOf-cW9zR4c25DLESQsOmhQExiNxd+pAixUqmX-KkSANjvjA@mail.gmail.com>
	<CAHut+PsHXoQRDXH0eyA7MLtJ8bCuEjtcfiX+2yR1N0S0KpDptA@mail.gmail.com>
	<CAJcOf-dE2vWpmqgEWj_tJ+i-oss_hhn9QjGpth+xTAyZUCANCQ@mail.gmail.com>
	<1947363.1601071129@sss.pgh.pa.us>
	<5708.1601145259@sss.pgh.pa.us>
	<CAJcOf-f8a-ZTsNRaNNJXFKnwtdbkgYuwjX342zL_+qmJwQB8bw@mail.gmail.com>
	<91749879-e086-02db-a4dd-44d49f0332b2@postgrespro.ru>

Anastasia Lubennikova <a.lubennikova@postgrespro.ru> writes:
> Hi, this entry is "Waiting on Author" and the thread was inactive for a 
> while. As far as I see, the patch needs some further work.
> Are you going to continue working on it, or should I mark it as 
> "returned with feedback" until a better time?

I'm inclined to go ahead and commit the 0001 patch I posted at [1]
(ie, change the implementation of GUC_REPORT to avoid intra-query
reports), since that addresses a performance problem that's
independent of the goal here.  The rest of this seems to still
be in Greg's court.

Has anyone got an opinion about the further improvement I suggested:

>> As it stands, 0001 reduces the ParameterStatus message traffic to
>> at most one per GUC per query, but it doesn't attempt to eliminate
>> duplicate ParameterStatus messages altogether.  We could do that
>> as a pretty simple adjustment if we're willing to expend the storage
>> to remember the last value sent to the client.  It might be worth
>> doing, since for example the function-SET-clause case would typically
>> lead to no net change in the GUC's value by the end of the query.

On reflection this seems worth doing, since excess client traffic
is far from free.

			regards, tom lane

[1] https://www.postgresql.org/message-id/5708.1601145259%40sss.pgh.pa.us





view thread (149+ messages)  latest in thread

Message-ID: <311787.1606254559@sss.pgh.pa.us>
Permalink:  ../311787.1606254559@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/311787.1606254559@sss.pgh.pa.us

 · 

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: pgsql-hackers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, a.lubennikova@postgrespro.ru, gregn4422@gmail.com, smithpb2250@gmail.com, peters@fast.au.fujitsu.com, david@pgmasters.net, tsunakawa.takay@fujitsu.com, alvherre@2ndquadrant.com, kommi.haribabu@gmail.com, robertmhaas@gmail.com, laurenz.albe@cybertec.at, pg@fastcrypt.com, jingwangian@gmail.com, elprans@gmail.com
  Subject: Re: Libpq support to connect to standby server as priority
  In-Reply-To: <311787.1606254559@sss.pgh.pa.us>

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

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