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

help / color / mirror / Atom feed
From: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: Tomas Vondra <tomas.vondra@2ndquadrant.com>
To: Erik Rijkers <er@xs4all.nl>
Cc: Masahiko Sawada <sawada.mshk@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions
Date: Sun, 1 Jul 2018 13:43:50 +0200
Message-ID: <e84013b6-b890-1c4c-2b82-eca185a2197c@2ndquadrant.com> (raw)
In-Reply-To: <5eddec05-404f-3fc4-bafd-15e9e4787705@2ndquadrant.com>
References: <688b0b7f-2f6c-d827-c27b-216a8e3ea700@2ndquadrant.com>
	<bbe1333f-17ba-1156-0713-7cc0bfe2d842@2ndquadrant.com>
	<bc83b847-2fc7-6561-4ef1-a3c9d154cc31@2ndquadrant.com>
	<47223121-222d-8e04-4928-a28ecdc5327d@2ndquadrant.com>
	<CAD21AoBa2mhbhZW6hnDRKs1ggycJOBE69eHk3KgCJ5Cwj9-ZkA@mail.gmail.com>
	<b9b306a3-c0a2-7eb9-92eb-a3ca111bd1bc@2ndquadrant.com>
	<4762f6bd-b147-cd06-f286-22f54790b77d@2ndquadrant.com>
	<cb2530b102afad854e051cca44577c6a@xs4all.nl>
	<d807dc44-55f9-5c23-97e5-a5881f4135e2@2ndquadrant.com>
	<5eddec05-404f-3fc4-bafd-15e9e4787705@2ndquadrant.com>

This patch set was not updated for the 2018-07 commitfest, so moved to -09.


On 09.03.18 17:07, Peter Eisentraut wrote:
> I think this patch is not going to be ready for PG11.
> 
> - It depends on some work in the thread "logical decoding of two-phase
> transactions", which is still in progress.
> 
> - Various details in the logical_work_mem patch (0001) are unresolved.
> 
> - This being partially a performance feature, we haven't seen any
> performance tests (e.g., which settings result in which latencies under
> which workloads).
> 
> That said, the feature seems useful and desirable, and the
> implementation makes sense.  There are documentation and tests.  But
> there is a significant amount of design and coding work still necessary.
> 
> Attached is a fixup patch that I needed to make it compile.
> 
> The last two patches in your series (0008, 0009) are labeled as bug
> fixes.  Would you like to argue that they should be applied
> independently of the rest of the feature?
> 


-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services




view thread (536+ messages)  latest in thread

Message-ID: <e84013b6-b890-1c4c-2b82-eca185a2197c@2ndquadrant.com>
Permalink:  ../e84013b6-b890-1c4c-2b82-eca185a2197c@2ndquadrant.com/
Also on:    postgresql.org/message-id/e84013b6-b890-1c4c-2b82-eca185a2197c@2ndquadrant.com

 · 

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: peter.eisentraut@2ndquadrant.com, tomas.vondra@2ndquadrant.com, er@xs4all.nl, sawada.mshk@gmail.com
  Subject: Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions
  In-Reply-To: <e84013b6-b890-1c4c-2b82-eca185a2197c@2ndquadrant.com>

* 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