pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Andres Freund <andres@anarazel.de>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Bruce Momjian <bruce@momjian.us>
Cc: Michael Paquier <michael.paquier@gmail.com>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Stephen Frost <sfrost@snowman.net>
Cc: Magnus Hagander <magnus@hagander.net>
Cc: pgsql-hackers@postgresql.org <pgsql-hackers@postgresql.org>
Cc: pgsql-core <pgsql-core@postgresql.org>
Subject: Re: [CORE] postpone next week's release
Date: Sun, 31 May 2015 22:16:29 +0200
Message-ID: <20150531201629.GA32602@awork2.anarazel.de> (raw)
In-Reply-To: <32433.1433087744@sss.pgh.pa.us>
References: <CA+TgmoYSO0a9YDe2bhpZoPgO4m62CyQL26LRkCQ8=vrQ8ZwdXA@mail.gmail.com>
	<12510.1432958887@sss.pgh.pa.us>
	<CA+TgmoYovkev0xXnxaTD8vnt=M9Cn+RmOkVEouqwj6_9JLkzLw@mail.gmail.com>
	<20150530154559.GA25053@momjian.us>
	<20150530204727.GJ13944@alap3.anarazel.de>
	<20150531024845.GB13697@momjian.us>
	<CAB7nPqQgitxp5505bSdNpGxOwrDebqF520BYLi=fJQLp0ZSd3A@mail.gmail.com>
	<20150531135025.GC13697@momjian.us>
	<20150531140305.GD13697@momjian.us>
	<32433.1433087744@sss.pgh.pa.us>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>

On 2015-05-31 11:55:44 -0400, Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> >> FYI, I realize that one additional thing that has discouraged code
> >> reorganization is the additional backpatch overhead.  I think we now
> >> need to accept that our reorganization-adverse approach might have cost
> >> us some reliability, and that reorganization is going to add work to
> >> backpatching.
> 
> > Actually, code reorganization in HEAD might cause backpatching to be
> > more buggy, reducing reliability --- obviously we need to have a
> > discussion about that.
> 
> Commit 6b700301c36e380eb4972ab72c0e914cae60f9fd is a recent real example.
> Not that that should dissuade us from ever doing any reorganizations,
> but it's foolish to discount back-patching costs.

On the other hand, that code is a complete maintenance nightmare. If
there weren't literally dozens of places that needed to be touched to
add a single parameter, it'd be far less likely for such a mistake to be
made. Right now significant portions of the file differ between the
branches, despite primarily minor feature additions...


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers



view thread (78+ messages)  latest in thread

Message-ID: <20150531201629.GA32602@awork2.anarazel.de>
Permalink:  ../20150531201629.GA32602@awork2.anarazel.de/
Also on:    postgresql.org/message-id/20150531201629.GA32602@awork2.anarazel.de

 · 

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: andres@anarazel.de, tgl@sss.pgh.pa.us, bruce@momjian.us, michael.paquier@gmail.com, robertmhaas@gmail.com, sfrost@snowman.net, magnus@hagander.net, pgsql-core@postgresql.org
  Subject: Re: [CORE] postpone next week's release
  In-Reply-To: <20150531201629.GA32602@awork2.anarazel.de>

* 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