Lists: | pgsql-hackers |
---|
From: | Radosław Smogura <rsmogura(at)softperience(dot)eu> |
---|---|
To: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Void binary patch |
Date: | 2011-02-20 10:20:22 |
Message-ID: | 201102201120.22178.rsmogura@softperience.eu |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Just patch for missing procedures for void send/recv
Regards,
Radek
Attachment | Content-Type | Size |
---|---|---|
void_binary.patch | text/x-patch | 3.1 KB |
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Radosław Smogura <rsmogura(at)softperience(dot)eu> |
Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-21 03:59:12 |
Message-ID: | 20110221035912.GB15510@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Sun, Feb 20, 2011 at 11:20:22AM +0100, Radosław Smogura wrote:
> Just patch for missing procedures for void send/recv
>
> Regards,
> Radek
Thanks! :)
Style note: the project doesn't use // as a comment-to-end-of-line in
C because some supported compilers don't understand it.
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Radosław Smogura <rsmogura(at)softperience(dot)eu> |
Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 12:01:02 |
Message-ID: | AANLkTikCgNt0+Q1SFr8GGbXmRqPWvTTVhU9zXzQZcf=V@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Sun, Feb 20, 2011 at 5:20 AM, Radosław Smogura
<rsmogura(at)softperience(dot)eu> wrote:
> Just patch for missing procedures for void send/recv
What problem does this fix?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From: | rsmogura <rsmogura(at)softperience(dot)eu> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 13:36:11 |
Message-ID: | faa88899104c10d46c886aadbecc35bf@mail.softperience.eu |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Tue, 22 Feb 2011 07:01:02 -0500, Robert Haas wrote:
> On Sun, Feb 20, 2011 at 5:20 AM, Radosław Smogura
> <rsmogura(at)softperience(dot)eu> wrote:
>> Just patch for missing procedures for void send/recv
>
> What problem does this fix?
Can not execute stored procedures in JDBC with out arguments, I think
function retuning void as well, and some other "minors". Ofc with binary
mode.
Regards,
Radek
From: | Merlin Moncure <mmoncure(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Radosław Smogura <rsmogura(at)softperience(dot)eu>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 14:12:23 |
Message-ID: | AANLkTikq562NXcuHEESTfxKKUU38k8=GUUcCMZZbGdN2@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Tue, Feb 22, 2011 at 6:01 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Sun, Feb 20, 2011 at 5:20 AM, Radosław Smogura
> <rsmogura(at)softperience(dot)eu> wrote:
>> Just patch for missing procedures for void send/recv
>
> What problem does this fix?
void returning functions may not be called when binary protocol is
requested currently. this is annoying: some drivers that wrap libpq
or the protocol directly use the binary mode exclusively and this
causes headaches for them. put another way, 'void' is the only POD
type missing send/recv.
merlin
From: | rsmogura <rsmogura(at)softperience(dot)eu> |
---|---|
To: | Merlin Moncure <mmoncure(at)gmail(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 14:22:56 |
Message-ID: | 5f95c1af27faedeca19c741083fc811d@mail.softperience.eu |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Tue, 22 Feb 2011 08:12:23 -0600, Merlin Moncure wrote:
> On Tue, Feb 22, 2011 at 6:01 AM, Robert Haas <robertmhaas(at)gmail(dot)com>
> wrote:
>> On Sun, Feb 20, 2011 at 5:20 AM, Radosław Smogura
>> <rsmogura(at)softperience(dot)eu> wrote:
>>> Just patch for missing procedures for void send/recv
>>
>> What problem does this fix?
>
> void returning functions may not be called when binary protocol is
> requested currently. this is annoying: some drivers that wrap libpq
> or the protocol directly use the binary mode exclusively and this
> causes headaches for them. put another way, 'void' is the only POD
> type missing send/recv.
>
> merlin
Just curious what POD means?
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Merlin Moncure <mmoncure(at)gmail(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Radosław Smogura <rsmogura(at)softperience(dot)eu>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 15:15:07 |
Message-ID: | 29479.1298387707@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Merlin Moncure <mmoncure(at)gmail(dot)com> writes:
> On Tue, Feb 22, 2011 at 6:01 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>> What problem does this fix?
> void returning functions may not be called when binary protocol is
> requested currently. this is annoying: some drivers that wrap libpq
> or the protocol directly use the binary mode exclusively and this
> causes headaches for them. put another way, 'void' is the only POD
> type missing send/recv.
Yeah, this has been discussed before.
Even though this patch is far past the CF deadline, I'm a bit tempted to
push it into 9.1 anyway, just so we can check off that problem.
regards, tom lane
From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Merlin Moncure <mmoncure(at)gmail(dot)com>, Radosław Smogura <rsmogura(at)softperience(dot)eu>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 16:31:22 |
Message-ID: | AANLkTi=WboSK12ZDCR3vC2wSeOfdaAQ9dOcPjOrtMVnK@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Tue, Feb 22, 2011 at 10:15 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Merlin Moncure <mmoncure(at)gmail(dot)com> writes:
>> On Tue, Feb 22, 2011 at 6:01 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>>> What problem does this fix?
>
>> void returning functions may not be called when binary protocol is
>> requested currently. this is annoying: some drivers that wrap libpq
>> or the protocol directly use the binary mode exclusively and this
>> causes headaches for them. put another way, 'void' is the only POD
>> type missing send/recv.
>
> Yeah, this has been discussed before.
>
> Even though this patch is far past the CF deadline, I'm a bit tempted to
> push it into 9.1 anyway, just so we can check off that problem.
+1.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From: | Merlin Moncure <mmoncure(at)gmail(dot)com> |
---|---|
To: | rsmogura <rsmogura(at)softperience(dot)eu> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 17:39:25 |
Message-ID: | AANLkTinTOyjnyPoTuUh9kZTQdpn2An06sY4vw7xdmyyO@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Tue, Feb 22, 2011 at 8:22 AM, rsmogura <rsmogura(at)softperience(dot)eu> wrote:
> On Tue, 22 Feb 2011 08:12:23 -0600, Merlin Moncure wrote:
>>
>> On Tue, Feb 22, 2011 at 6:01 AM, Robert Haas <robertmhaas(at)gmail(dot)com>
>> wrote:
>>>
>>> On Sun, Feb 20, 2011 at 5:20 AM, Radosław Smogura
>>> <rsmogura(at)softperience(dot)eu> wrote:
>>>>
>>>> Just patch for missing procedures for void send/recv
>>>
>>> What problem does this fix?
>>
>> void returning functions may not be called when binary protocol is
>> requested currently. this is annoying: some drivers that wrap libpq
>> or the protocol directly use the binary mode exclusively and this
>> causes headaches for them. put another way, 'void' is the only POD
>> type missing send/recv.
>>
>> merlin
>
> Just curious what POD means?
POD = 'Plain Old Data' -- one of the core types. :-).
merlin
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Merlin Moncure <mmoncure(at)gmail(dot)com>, Radosław Smogura <rsmogura(at)softperience(dot)eu>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Void binary patch |
Date: | 2011-02-22 18:09:24 |
Message-ID: | 5326.1298398164@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Tue, Feb 22, 2011 at 10:15 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Yeah, this has been discussed before.
>>
>> Even though this patch is far past the CF deadline, I'm a bit tempted to
>> push it into 9.1 anyway, just so we can check off that problem.
> +1.
Done.
regards, tom lane