Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hMB9a-00015L-B3 for pgsql-hackers@arkaria.postgresql.org; Thu, 02 May 2019 12:49:34 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1hMB9Y-0006Bc-UF for pgsql-hackers@arkaria.postgresql.org; Thu, 02 May 2019 12:49:32 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hMB9Y-00063f-Iy for pgsql-hackers@lists.postgresql.org; Thu, 02 May 2019 12:49:32 +0000 Received: from mx2a.mailbox.org ([2001:67c:2050:104:0:2:25:2] helo=mx2.mailbox.org) by magus.postgresql.org with esmtps (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hMB9W-0004Yh-Ft for pgsql-hackers@lists.postgresql.org; Thu, 02 May 2019 12:49:32 +0000 Received: from smtp1.mailbox.org (smtp1.mailbox.org [IPv6:2001:67c:2050:105:465:1:1:0]) (using TLSv1.2 with cipher ECDHE-RSA-CHACHA20-POLY1305 (256/256 bits)) (No client certificate requested) by mx2.mailbox.org (Postfix) with ESMTPS id 4325DA10D5 for ; Thu, 2 May 2019 14:49:28 +0200 (CEST) X-Virus-Scanned: amavisd-new at heinlein-support.de Received: from smtp1.mailbox.org ([80.241.60.240]) by hefe.heinlein-support.de (hefe.heinlein-support.de [91.198.250.172]) (amavisd-new, port 10030) with ESMTP id U2cRItYujePs for ; Thu, 2 May 2019 14:49:24 +0200 (CEST) Message-ID: <539fbfe416eb3d7ebf72268eb839163a68aed9ef.camel@cybertec.at> Subject: Re: Bad canonicalization for dateranges with 'infinity' bounds From: Laurenz Albe To: pgsql-hackers@lists.postgresql.org Date: Thu, 02 May 2019 14:49:23 +0200 In-Reply-To: <77f24ea19ab802bc9bc60ddbb8977ee2d646aec1.camel@cybertec.at> References: <77f24ea19ab802bc9bc60ddbb8977ee2d646aec1.camel@cybertec.at> Organization: Cybertec Content-Type: multipart/mixed; boundary="=-033TniUsPsCPGJiVf7PA" MIME-Version: 1.0 List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk --=-033TniUsPsCPGJiVf7PA Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit I wrote: > I propose the attached patch which fixes the problem. I forgot to attach the patch. Here it is. Yours, Laurenz Albe --=-033TniUsPsCPGJiVf7PA Content-Disposition: attachment; filename="0001-Don-t-canonicalize-dateranges-with-infinity-bounds.patch" Content-Type: text/x-patch; name="0001-Don-t-canonicalize-dateranges-with-infinity-bounds.patch"; charset="UTF-8" Content-Transfer-Encoding: base64 RnJvbSA2YmJhZDBhY2YzYmFhZTNhMDhkMWY5MTFiNzAxNzY0MmM4YThhZmU5IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBMYXVyZW56IEFsYmUgPGxhdXJlbnouYWxiZUBjeWJlcnRlYy5h dD4KRGF0ZTogVGh1LCAyIE1heSAyMDE5IDE0OjMyOjI3ICswMjAwClN1YmplY3Q6IFtQQVRDSF0g RG9uJ3QgY2Fub25pY2FsaXplIGRhdGVyYW5nZXMgd2l0aCAnaW5maW5pdHknIGJvdW5kcwoKU2lu Y2UgYWRkaW5nIG9uZSB0byBpbmZpbml0eSBkb2Vzbid0IGNoYW5nZSB0aGUgdmFsdWUsIGNhbm9u aWNhbGl6YXRpb24Kb2YgZGF0ZXJhbmdlcyB3aXRoIGFuIGV4cGxpY2l0ICctaW5maW5pdHknIGFz IGV4Y2x1c2l2ZSBsb3dlciBib3VuZAp3b3VsZCBtYWtlIGl0IGFuIGluY2x1c2l2ZSBib3VuZCwg d2hpbGUgZGF0ZXJhbmdlcyB3aXRoICdpbmZpbml0eScgYXMKaW5jbHVzaXZlIHVwcGVyIGJvdW5k IHdvdWxkIHR1cm4gaXQgaW50byBhbiBleGNsdXNpdmUgYm91bmQuCi0tLQogc3JjL2JhY2tlbmQv dXRpbHMvYWR0L3JhbmdldHlwZXMuYyAgICAgICB8ICA0ICsrLS0KIHNyYy90ZXN0L3JlZ3Jlc3Mv ZXhwZWN0ZWQvcmFuZ2V0eXBlcy5vdXQgfCAyNCArKysrKysrKysrKysrKysrKysrKysrKysKIHNy Yy90ZXN0L3JlZ3Jlc3Mvc3FsL3JhbmdldHlwZXMuc3FsICAgICAgfCAgNCArKysrCiAzIGZpbGVz IGNoYW5nZWQsIDMwIGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEv c3JjL2JhY2tlbmQvdXRpbHMvYWR0L3JhbmdldHlwZXMuYyBiL3NyYy9iYWNrZW5kL3V0aWxzL2Fk dC9yYW5nZXR5cGVzLmMKaW5kZXggNzJjNDUwYzcwZS4uYTk4ZjE3YmI2NiAxMDA2NDQKLS0tIGEv c3JjL2JhY2tlbmQvdXRpbHMvYWR0L3JhbmdldHlwZXMuYworKysgYi9zcmMvYmFja2VuZC91dGls cy9hZHQvcmFuZ2V0eXBlcy5jCkBAIC0xNDMxLDEzICsxNDMxLDEzIEBAIGRhdGVyYW5nZV9jYW5v bmljYWwoUEdfRlVOQ1RJT05fQVJHUykKIAlpZiAoZW1wdHkpCiAJCVBHX1JFVFVSTl9SQU5HRV9Q KHIpOwogCi0JaWYgKCFsb3dlci5pbmZpbml0ZSAmJiAhbG93ZXIuaW5jbHVzaXZlKQorCWlmICgh KGxvd2VyLmluZmluaXRlIHx8IERBVEVfTk9UX0ZJTklURShsb3dlci52YWwpKSAmJiAhbG93ZXIu aW5jbHVzaXZlKQogCXsKIAkJbG93ZXIudmFsID0gRGlyZWN0RnVuY3Rpb25DYWxsMihkYXRlX3Bs aSwgbG93ZXIudmFsLCBJbnQzMkdldERhdHVtKDEpKTsKIAkJbG93ZXIuaW5jbHVzaXZlID0gdHJ1 ZTsKIAl9CiAKLQlpZiAoIXVwcGVyLmluZmluaXRlICYmIHVwcGVyLmluY2x1c2l2ZSkKKwlpZiAo ISh1cHBlci5pbmZpbml0ZSB8fCBEQVRFX05PVF9GSU5JVEUodXBwZXIudmFsKSkgJiYgdXBwZXIu aW5jbHVzaXZlKQogCXsKIAkJdXBwZXIudmFsID0gRGlyZWN0RnVuY3Rpb25DYWxsMihkYXRlX3Bs aSwgdXBwZXIudmFsLCBJbnQzMkdldERhdHVtKDEpKTsKIAkJdXBwZXIuaW5jbHVzaXZlID0gZmFs c2U7CmRpZmYgLS1naXQgYS9zcmMvdGVzdC9yZWdyZXNzL2V4cGVjdGVkL3JhbmdldHlwZXMub3V0 IGIvc3JjL3Rlc3QvcmVncmVzcy9leHBlY3RlZC9yYW5nZXR5cGVzLm91dAppbmRleCBhY2NmMWUw ZDllLi42MGQ4NzVlODk4IDEwMDY0NAotLS0gYS9zcmMvdGVzdC9yZWdyZXNzL2V4cGVjdGVkL3Jh bmdldHlwZXMub3V0CisrKyBiL3NyYy90ZXN0L3JlZ3Jlc3MvZXhwZWN0ZWQvcmFuZ2V0eXBlcy5v dXQKQEAgLTY1Miw2ICs2NTIsMzAgQEAgc2VsZWN0IGRhdGVyYW5nZSgnMjAwMC0wMS0xMCc6OmRh dGUsICcyMDAwLTAxLTExJzo6ZGF0ZSwgJyhdJyk7CiAgWzAxLTExLTIwMDAsMDEtMTItMjAwMCkK ICgxIHJvdykKIAorc2VsZWN0IGRhdGVyYW5nZSgnLWluZmluaXR5Jzo6ZGF0ZSwgJzIwMDAtMDEt MDEnOjpkYXRlLCAnKCknKTsKKyAgICAgICBkYXRlcmFuZ2UgICAgICAgIAorLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tCisgKC1pbmZpbml0eSwwMS0wMS0yMDAwKQorKDEgcm93KQorCitzZWxlY3Qg ZGF0ZXJhbmdlKCctaW5maW5pdHknOjpkYXRlLCAnMjAwMC0wMS0wMSc6OmRhdGUsICdbKScpOwor ICAgICAgIGRhdGVyYW5nZSAgICAgICAgCistLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KKyBbLWlu ZmluaXR5LDAxLTAxLTIwMDApCisoMSByb3cpCisKK3NlbGVjdCBkYXRlcmFuZ2UoJzIwMDAtMDEt MDEnOjpkYXRlLCAnaW5maW5pdHknOjpkYXRlLCAnWyknKTsKKyAgICAgICBkYXRlcmFuZ2UgICAg ICAgCistLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQorIFswMS0wMS0yMDAwLGluZmluaXR5KQorKDEg cm93KQorCitzZWxlY3QgZGF0ZXJhbmdlKCcyMDAwLTAxLTAxJzo6ZGF0ZSwgJ2luZmluaXR5Jzo6 ZGF0ZSwgJ1tdJyk7CisgICAgICAgZGF0ZXJhbmdlICAgICAgIAorLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0KKyBbMDEtMDEtMjAwMCxpbmZpbml0eV0KKygxIHJvdykKKwogLS0gdGVzdCBHaVNUIGlu ZGV4IHRoYXQncyBiZWVuIGJ1aWx0IGluY3JlbWVudGFsbHkKIGNyZWF0ZSB0YWJsZSB0ZXN0X3Jh bmdlX2dpc3QoaXIgaW50NHJhbmdlKTsKIGNyZWF0ZSBpbmRleCB0ZXN0X3JhbmdlX2dpc3RfaWR4 IG9uIHRlc3RfcmFuZ2VfZ2lzdCB1c2luZyBnaXN0IChpcik7CmRpZmYgLS1naXQgYS9zcmMvdGVz dC9yZWdyZXNzL3NxbC9yYW5nZXR5cGVzLnNxbCBiL3NyYy90ZXN0L3JlZ3Jlc3Mvc3FsL3Jhbmdl dHlwZXMuc3FsCmluZGV4IDU1NjM4YTg1ZWUuLjlmZGIxOTUzZGYgMTAwNjQ0Ci0tLSBhL3NyYy90 ZXN0L3JlZ3Jlc3Mvc3FsL3JhbmdldHlwZXMuc3FsCisrKyBiL3NyYy90ZXN0L3JlZ3Jlc3Mvc3Fs L3JhbmdldHlwZXMuc3FsCkBAIC0xNjUsNiArMTY1LDEwIEBAIHNlbGVjdCBkYXRlcmFuZ2UoJzIw MDAtMDEtMTAnOjpkYXRlLCAnMjAwMC0wMS0yMCc6OmRhdGUsICcoXScpOwogc2VsZWN0IGRhdGVy YW5nZSgnMjAwMC0wMS0xMCc6OmRhdGUsICcyMDAwLTAxLTIwJzo6ZGF0ZSwgJygpJyk7CiBzZWxl Y3QgZGF0ZXJhbmdlKCcyMDAwLTAxLTEwJzo6ZGF0ZSwgJzIwMDAtMDEtMTEnOjpkYXRlLCAnKCkn KTsKIHNlbGVjdCBkYXRlcmFuZ2UoJzIwMDAtMDEtMTAnOjpkYXRlLCAnMjAwMC0wMS0xMSc6OmRh dGUsICcoXScpOworc2VsZWN0IGRhdGVyYW5nZSgnLWluZmluaXR5Jzo6ZGF0ZSwgJzIwMDAtMDEt MDEnOjpkYXRlLCAnKCknKTsKK3NlbGVjdCBkYXRlcmFuZ2UoJy1pbmZpbml0eSc6OmRhdGUsICcy MDAwLTAxLTAxJzo6ZGF0ZSwgJ1spJyk7CitzZWxlY3QgZGF0ZXJhbmdlKCcyMDAwLTAxLTAxJzo6 ZGF0ZSwgJ2luZmluaXR5Jzo6ZGF0ZSwgJ1spJyk7CitzZWxlY3QgZGF0ZXJhbmdlKCcyMDAwLTAx LTAxJzo6ZGF0ZSwgJ2luZmluaXR5Jzo6ZGF0ZSwgJ1tdJyk7CiAKIC0tIHRlc3QgR2lTVCBpbmRl eCB0aGF0J3MgYmVlbiBidWlsdCBpbmNyZW1lbnRhbGx5CiBjcmVhdGUgdGFibGUgdGVzdF9yYW5n ZV9naXN0KGlyIGludDRyYW5nZSk7Ci0tIAoyLjIwLjEKCg== --=-033TniUsPsCPGJiVf7PA--