Thread: dblink: add polymorphic functions - review

dblink: add polymorphic functions - review

From
Pavel Stehule
Date:
Hi

I am trying to check your patch - but diffs that you sent to mailing list are inconsistent.

Please, can you send one complete patch?

Regards

Pavel

Re: dblink: add polymorphic functions - review

From
Michael Paquier
Date:
On Thu, Jun 18, 2015 at 10:44 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> Hi
>
> I am trying to check your patch - but diffs that you sent to mailing list
> are inconsistent.
>
> Please, can you send one complete patch?

In order to follow more easily the thread of this patch, could you
answer directly to the existing thread? FWIW, it is here:

http://www.postgresql.org/message-id/flat/CADkLM=d9AEZYQ2TpzOJQwBb42nV49YQy6b6S=z4q9svJiQLB-g@mail.gmail.com#CADkLM=d9AEZYQ2TpzOJQwBb42nV49YQy6b6S=z4q9svJiQLB-g@mail.gmail.com

This will make the life of the future CFM(s) easier.
Regards,
-- 
Michael



Re: dblink: add polymorphic functions - review

From
Pavel Stehule
Date:


2015-06-19 2:17 GMT+02:00 Michael Paquier <michael.paquier@gmail.com>:
On Thu, Jun 18, 2015 at 10:44 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> Hi
>
> I am trying to check your patch - but diffs that you sent to mailing list
> are inconsistent.
>
> Please, can you send one complete patch?

In order to follow more easily the thread of this patch, could you
answer directly to the existing thread? FWIW, it is here:
http://www.postgresql.org/message-id/flat/CADkLM=d9AEZYQ2TpzOJQwBb42nV49YQy6b6S=z4q9svJiQLB-g@mail.gmail.com#CADkLM=d9AEZYQ2TpzOJQwBb42nV49YQy6b6S=z4q9svJiQLB-g@mail.gmail.com

This will make the life of the future CFM(s) easier.
Regards,

I am sorry. I didn't the original mail in my mailbox.

Regards

Pavel
 
--
Michael

Re: dblink: add polymorphic functions - review

From
Thomas Munro
Date:
On Fri, Jun 19, 2015 at 4:17 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> I am sorry. I didn't the original mail in my mailbox.

There is a (rather well hidden) way to ask Majordomo to send you a
message that arrived before you subscribed, so that you can reply
without creating a new thread.  I think you have to subscribe first so
that you can log into it.  If you start here and browse to the message
in the archive, you'll see a 'Mail this message to ...' link at the
bottom:

https://lists.postgresql.org/mj/mj_wwwusr/domain=postgresql.org

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



Re: dblink: add polymorphic functions - review

From
Pavel Stehule
Date:


2015-06-19 6:32 GMT+02:00 Thomas Munro <thomas.munro@enterprisedb.com>:
On Fri, Jun 19, 2015 at 4:17 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
> I am sorry. I didn't the original mail in my mailbox.

There is a (rather well hidden) way to ask Majordomo to send you a
message that arrived before you subscribed, so that you can reply
without creating a new thread.  I think you have to subscribe first so
that you can log into it.  If you start here and browse to the message
in the archive, you'll see a 'Mail this message to ...' link at the
bottom:

https://lists.postgresql.org/mj/mj_wwwusr/domain=postgresql.org


I didn't know it. Thank you

Regards

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

Re: dblink: add polymorphic functions - review

From
Joe Conway
Date:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 06/18/2015 09:57 PM, Pavel Stehule wrote:
> 2015-06-19 6:32 GMT+02:00 Thomas Munro
> <thomas.munro@enterprisedb.com 
> <mailto:thomas.munro@enterprisedb.com>>:
> 
> On Fri, Jun 19, 2015 at 4:17 PM, Pavel Stehule 
> <pavel.stehule@gmail.com <mailto:pavel.stehule@gmail.com>> wrote:
>> I am sorry. I didn't the original mail in my mailbox.
> 
> There is a (rather well hidden) way to ask Majordomo to send you a 
> message that arrived before you subscribed, so that you can reply 
> without creating a new thread.  I think you have to subscribe first
> so that you can log into it.  If you start here and browse to the
> message in the archive, you'll see a 'Mail this message to ...'
> link at the bottom:
> 
> https://lists.postgresql.org/mj/mj_wwwusr/domain=postgresql.org
> 
> I didn't know it. Thank you

Nice tip -- thanks! The patch was missing two new files - I posted a
complete patch using the original thread.


- -- 
Joe Conway
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)

iQIcBAEBAgAGBQJVmYWDAAoJEDfy90M199hlT7YP/1rUX6TTJcTT45bRPWagbPSa
VS6NBihFDA4Kc6IDfvr7iS3nyWaHC1TDBrkhHKUfH1HSxXN492GkLCjG95HCSEHQ
AFylW3IarVpMdCxe7qJ2lH1CG69v8pabfmVPleymNZqz6j3KlNDXML9OqYjv+Aqx
G7Z6ciKC673+wQN6g4u9i8bbgJwHmqCdwEGLKgGqNnwErpNyTpxpr3tHZrkAC91B
KY6MBrNkpKyFbhxl5+1CIU4q6CWcYMtQSQ5T8eqKk2sntdHc3iDtrMFeSkK3OBRy
vbfaYrHRpFfk3OnBHNDlmzV0N8zVnGHrSgIkqA2tbFJ9ZMhT1AJavCrjBNeuiuMN
WVzpj+KY7lwbLI4xl3w+EUyqpnheVkNLuEit87EzIxiA2Sus1+XLFpPzE6O9pUN8
olN452ZYP5cjlApKxiEn0Bo0aEe7a+VpUja4/Z6dq3A7V42ci/HKNLwiI2n2iQQn
AiKM91Y3ir3FEcBg67CIbZdjxrrbwrzsMSfRi06INQIKTswMIlDR2A+2wVEtrDe5
kjqgVA1+bXfjlN2lfN9bp5KArXnvRLg+WA9Jj7SLB0I3QbiDgBqC7DS2ght0FxHp
4bY7hIL+kE8dc8vPs8C83WC8LYXxqFzi9vgZDk5kyqNTrQfLMz5YbPDiCQPUybJf
u9nNrqPmS+0BKOSr5o36
=orOZ
-----END PGP SIGNATURE-----