Received: from malur.postgresql.org ([2a02:16a8:dc51::56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1flj21-000737-7T for pgsql-bugs@arkaria.postgresql.org; Fri, 03 Aug 2018 22:58:50 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1flj1z-0002rf-U1 for pgsql-bugs@arkaria.postgresql.org; Fri, 03 Aug 2018 22:58:47 +0000 Received: from makus.postgresql.org ([2001:4800:1501:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1flj1z-0002rY-Kl for pgsql-bugs@lists.postgresql.org; Fri, 03 Aug 2018 22:58:47 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.89) (envelope-from ) id 1flj1x-0004B9-4f for pgsql-bugs@lists.postgresql.org; Fri, 03 Aug 2018 22:58:46 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.4/8.14.4) with ESMTP id w73MwdI9008969; Fri, 3 Aug 2018 18:58:39 -0400 From: Tom Lane To: Andres Freund cc: Dean Rasheed , Amit Langote , Mario De Frutos Dieguez , pgsql-bugs@lists.postgresql.org Subject: Re: Fwd: Problem with a "complex" upsert In-reply-to: <20180803224455.4g2hnvciynu2s7iv@alap3.anarazel.de> References: <18795.1529600753@sss.pgh.pa.us> <8da3739d-da24-1125-cab6-b68d6ec1bc6a@lab.ntt.co.jp> <30415.1531256373@sss.pgh.pa.us> <4b956567-8283-1b8a-7cd7-f8b6e626bba9@lab.ntt.co.jp> <20180803184638.gdii7mja7qfc3fba@alap3.anarazel.de> <7664.1533335577@sss.pgh.pa.us> <20180803224455.4g2hnvciynu2s7iv@alap3.anarazel.de> Comments: In-reply-to Andres Freund message dated "Fri, 03 Aug 2018 15:44:55 -0700" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <8967.1533337119.1@sss.pgh.pa.us> Date: Fri, 03 Aug 2018 18:58:39 -0400 Message-ID: <8968.1533337119@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk Andres Freund writes: > On 2018-08-03 18:32:57 -0400, Tom Lane wrote: >> Balance the risks of shipping a broken release vs. waiting one >> more quarter to ship the fix. After a quick look at the size >> of the patch, my own inclination if I were the committer would >> be to wait till after the releases are out. Or you might >> consider pushing only to 11+HEAD, with the expectation of >> back-patching later after we've gotten some beta results. > This results in clearly inserting wrong data and/or crashing the > server. And it's not a huge effect outside of already broken scenarios. > I think we definitely should try to get this in. Well, if you're excited about it, help Dean review it. My own feeling is that this case has been broken for several years with no one noticing, so it can't be all that critical. regards, tom lane