Re: Covering GiST indexes - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: Covering GiST indexes
Date
Msg-id CAEepm=1zmt66h7=UE_ZgrA-az5mWgpgT9B5XeXD+kQgS0vqs8A@mail.gmail.com
Whole thread Raw
In response to Re: Covering GiST indexes  (Andrey Borodin <x4mmm@yandex-team.ru>)
Responses Re: Covering GiST indexes  (Andrey Borodin <x4mmm@yandex-team.ru>)
List pgsql-hackers
On Sun, Jul 29, 2018 at 10:50 PM, Andrey Borodin <x4mmm@yandex-team.ru> wrote:
> Thanks! The problem appeared with commit 701fd0b [0] which dropped
> validation rules checked in failed test.  Here's the patch with fixed tests.

Thanks.  I received the attachment.

Just as an FYI, something about the way your mail client encoded it
prevented the mail archives from picking it up (according to someone
who knows much more about these things than me, it's buried in the
"multipart/mixed" part, so not visible to the mail archive if it
extracts only the "text/plain" part, or something like that).  That
didn't happen on any of your other patches.  I have no opinion of
whether that's a bug in the mail archive or an unhelpful mail client
or a non-ideal way to post patches, but you can see here that we don't
have the patch in the usual place:

https://www.postgresql.org/message-id/850AE105-F89A-42E4-AD40-FBC6EA5A5A00%40yandex-team.ru

That explains why cfbot didn't automatically test your new patch, so
it still show as broken here:
http://cfbot.cputube.org/andrey-borodin.html

-- 
Thomas Munro
http://www.enterprisedb.com


pgsql-hackers by date:

Previous
From: "Bossart, Nathan"
Date:
Subject: Re: Add SKIP LOCKED to VACUUM and ANALYZE
Next
From: Tatsuo Ishii
Date:
Subject: Adding a note to protocol.sgml regarding CopyData