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

help / color / mirror / Atom feed
From: David Steele <david@pgmasters.net>
To: Andres Freund <andres@anarazel.de>
To: Tomas Vondra <tomas.vondra@2ndquadrant.com>
Cc: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions
Date: Fri, 2 Mar 2018 20:37:26 -0500
Message-ID: <ecc17b46-c2e6-5ed4-eb16-eb6bcb0908bd@pgmasters.net> (raw)
In-Reply-To: <20180303010153.myd3f2bcdvqo6ry5@alap3.anarazel.de>
References: <5d00346a-b565-7fde-c6da-35f39c72936a@2ndquadrant.com>
	<d72caaf1-a2da-081c-57ea-d30e5f47fd1e@2ndquadrant.com>
	<4d7ed7b8-df43-9b79-b04e-ad4c43ef519c@2ndquadrant.com>
	<d5f226c6-8a45-a9a3-4cb8-6d695b908eae@2ndquadrant.com>
	<ee4278d9-e421-145c-51d8-790a43ebff3d@2ndquadrant.com>
	<20180302011235.k7bxqmktrvgz5tpb@alap3.anarazel.de>
	<e976e0b4-e752-30f9-2025-95282f8f3ba8@2ndquadrant.com>
	<90964101-eef1-68b1-a337-5e5d4bd6e7f0@pgmasters.net>
	<20180302200529.fwiquub5cr3o4gpz@alap3.anarazel.de>
	<f288c9a2-fbad-6245-c061-90042f2d63ea@2ndquadrant.com>
	<20180303010153.myd3f2bcdvqo6ry5@alap3.anarazel.de>

On 3/2/18 8:01 PM, Andres Freund wrote:
> On 2018-03-03 02:00:46 +0100, Tomas Vondra wrote:
>> That is somewhat misleading, I think. You're right the last version was
>> submitted on 2018-01-19, but the next review arrived on 2018-01-31, i.e.
>> right at the end of the CF. So it's not like the patch was sitting there
>> with unresolved issues. Based on that review the patch was marked as RWF
>> and thus not moved to 2018-03 automatically.
> 
> I don't see how this changes anything.

I agree that things could be clearer, and Andres has produced a great
document that we can build on.  The old one had gotten a bit stale.

However, I think it's pretty obvious that a CF entry should be
accompanied with a patch.  It sounds like the timing was awkward but you
still had 28 days to produce a new patch.

I also notice that you submitted 7 patches in this CF but are reviewing
zero.

-- 
-David
david@pgmasters.net




view thread (536+ messages)  latest in thread

Message-ID: <ecc17b46-c2e6-5ed4-eb16-eb6bcb0908bd@pgmasters.net>
Permalink:  ../ecc17b46-c2e6-5ed4-eb16-eb6bcb0908bd@pgmasters.net/
Also on:    postgresql.org/message-id/ecc17b46-c2e6-5ed4-eb16-eb6bcb0908bd@pgmasters.net

 · 

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: david@pgmasters.net, andres@anarazel.de, tomas.vondra@2ndquadrant.com, peter.eisentraut@2ndquadrant.com
  Subject: Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions
  In-Reply-To: <ecc17b46-c2e6-5ed4-eb16-eb6bcb0908bd@pgmasters.net>

* 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