Re: (forw) [thomas.munro@enterprisedb.com: Re: Covering GiST indexes]
| От | Magnus Hagander |
|---|---|
| Тема | Re: (forw) [thomas.munro@enterprisedb.com: Re: Covering GiST indexes] |
| Дата | |
| Msg-id | CABUevEzpyoeGLns5cbXyh9ViVPZBG3K_B6O9s=rm2X0=5S7w4g@mail.gmail.com обсуждение исходный текст |
| Ответ на | (forw) [thomas.munro@enterprisedb.com: Re: Covering GiST indexes] (Stephen Frost <sfrost@snowman.net>) |
| Список | pgsql-www |
Actualkly, if I read it correct:
1. The email is mime multipart/alternative, with two parts (which tells us to use one of the two parts, not both)
2. Part 1 is the email in plaintext format.
3. Part 2 is in itself made out of three parts
3a. The mail in HTML format
3b. The attachment in application/octet-stream
3c. An empty piece of HTML.
In the second attempt to send the email, it has 3 simple mime parts (in multipart/mixed, so we look at all of them):
1. the email in text format
2. the attachment
3. an empty text part
Which rhymes well with what Thomas found out.
Do we know what the difference was in the two formats used to send it? Since clearly the MIME structure produced by Apple Mail is completely different.
//Magnus
On Mon, Jul 30, 2018 at 1:38 AM, Stephen Frost <sfrost@snowman.net> wrote:
Greetings,
Looks like there's an issue when it comes to handling attachements sent
via Apple Mail in the archives. Sending to -www and CC'ing Magnus for
his thoughts (though it might be a bit before we hear from him).
I've seen this also with posts Cynthia has made (she's now using Gmail
instead), so if there's any need to re-test or do something different
client-side, we can help there.
Thanks!
Stephen
----- Forwarded message from Thomas Munro <thomas.munro@enterprisedb.com> -----
Date: Mon, 30 Jul 2018 10:58:39 +1200
From: Thomas Munro <thomas.munro@enterprisedb.com>
To: Andrey Borodin <x4mmm@yandex-team.ru>
Cc: Teodor Sigaev <teodor@sigaev.ru>, pgsql-hackers <pgsql-hackers@postgresql.org>
Message-ID: <CAEepm=1zmt66h7=UE_ZgrA-az5mWgpgT9B5XeXD+kQgS0vqs8A@ mail.gmail.com>
X-Spam-Status: No, score=-7.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_ LIST_MULTI,RCVD_IN_DNSWL_HI, SPF_PASS,T_DKIMWL_WL_MED
autolearn=ham autolearn_force=no version=3.4.1
Subject: Re: Covering GiST indexes
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
----- End forwarded message -----
В списке pgsql-www по дате отправления: