On Sun, Jul 29, 2018 at 10:50 PM, Andrey Borodin <x4...@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