Lists: | pgsql-patches |
---|
From: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | actualized SQL/PSM patch |
Date: | 2008-03-04 20:32:27 |
Message-ID: | 162867790803041232u12dea812x88fd3a683f4c2a0c@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Hello
I actualized sql/psm patch. This patch can be downloaded from
http://www.pgsql.cz/patches/plpgpsm.diff.gz
Documentation is on wiki http://www.pgsql.cz/index.php/SQL/PSM_Manual
Regards
Pavel Stehule
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-03-05 03:44:37 |
Message-ID: | 200803050344.m253ibr14901@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Your patch has been added to the PostgreSQL unapplied patches list at:
http://momjian.postgresql.org/cgi-bin/pgpatches
It will be applied as soon as one of the PostgreSQL committers reviews
and approves it.
---------------------------------------------------------------------------
Pavel Stehule wrote:
> Hello
>
> I actualized sql/psm patch. This patch can be downloaded from
> http://www.pgsql.cz/patches/plpgpsm.diff.gz
>
> Documentation is on wiki http://www.pgsql.cz/index.php/SQL/PSM_Manual
>
> Regards
> Pavel Stehule
>
> --
> Sent via pgsql-patches mailing list (pgsql-patches(at)postgresql(dot)org)
> To make changes to your Subscription:
> http://mail.postgresql.org/mj/mj_wwwusr?domain=postgresql.org&extra=pgsql-patches
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-01 21:55:45 |
Message-ID: | 15285.1207086945@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
"Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> writes:
> I actualized sql/psm patch. This patch can be downloaded from
> http://www.pgsql.cz/patches/plpgpsm.diff.gz
The fundamental problem I've got with this patch is that it adds 400K
of new code (and that's just the code, not counting documentation or
regression tests) that we'll have to maintain, to obtain a feature that
so far as I've heard there is precisely zero demand for.
The duplicativeness of the code with plpgsql doesn't make this prospect
any more pleasant, either.
The idea would be a lot easier to swallow if the code were refactored
to avoid the duplication with plpgsql.
regards, tom lane
From: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-02 04:54:18 |
Message-ID: | 162867790804012154j77ce0094vec62d97c43346315@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Hello
On 01/04/2008, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> writes:
> > I actualized sql/psm patch. This patch can be downloaded from
> > http://www.pgsql.cz/patches/plpgpsm.diff.gz
>
> The fundamental problem I've got with this patch is that it adds 400K
> of new code (and that's just the code, not counting documentation or
> regression tests) that we'll have to maintain, to obtain a feature that
> so far as I've heard there is precisely zero demand for.
>
> The duplicativeness of the code with plpgsql doesn't make this prospect
> any more pleasant, either.
>
> The idea would be a lot easier to swallow if the code were refactored
> to avoid the duplication with plpgsql.
>
This is long run and needs hard reorganisation of plpgsql code. And
moving some plpgsql code to core. But I don't expect so plpgpsm code
can be less than 200KB.
Regards
Pavel Stehule
> regards, tom lane
>
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 00:44:32 |
Message-ID: | 200804030044.m330iWu21873@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
The author has received feedback so this has been saved for the next
commit-fest:
http://momjian.postgresql.org/cgi-bin/pgpatches_hold
---------------------------------------------------------------------------
Pavel Stehule wrote:
> Hello
>
> On 01/04/2008, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> writes:
> > > I actualized sql/psm patch. This patch can be downloaded from
> > > http://www.pgsql.cz/patches/plpgpsm.diff.gz
> >
> > The fundamental problem I've got with this patch is that it adds 400K
> > of new code (and that's just the code, not counting documentation or
> > regression tests) that we'll have to maintain, to obtain a feature that
> > so far as I've heard there is precisely zero demand for.
> >
> > The duplicativeness of the code with plpgsql doesn't make this prospect
> > any more pleasant, either.
> >
> > The idea would be a lot easier to swallow if the code were refactored
> > to avoid the duplication with plpgsql.
> >
>
> This is long run and needs hard reorganisation of plpgsql code. And
> moving some plpgsql code to core. But I don't expect so plpgpsm code
> can be less than 200KB.
>
> Regards
> Pavel Stehule
>
> > regards, tom lane
> >
>
> --
> Sent via pgsql-patches mailing list (pgsql-patches(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-patches
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From: | "Jonah H(dot) Harris" <jonah(dot)harris(at)gmail(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 02:06:46 |
Message-ID: | 36e682920804021906x2af73d5cyeda277d7a16b4fdf@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
On Tue, Apr 1, 2008 at 5:55 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> The fundamental problem I've got with this patch is that it adds 400K
> of new code (and that's just the code, not counting documentation or
> regression tests) that we'll have to maintain, to obtain a feature that
> so far as I've heard there is precisely zero demand for.
We have a customer that wants to use it as part of a MySQL-to-Postgres
migration.
--
Jonah H. Harris, Sr. Software Architect | phone: 732.331.1324
EnterpriseDB Corporation | fax: 732.331.1301
499 Thornall Street, 2nd Floor | jonah(dot)harris(at)enterprisedb(dot)com
Edison, NJ 08837 | http://www.enterprisedb.com/
From: | "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 02:34:00 |
Message-ID: | 20080402193400.3537d601@commandprompt.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Tue, 01 Apr 2008 17:55:45 -0400
Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> writes:
> > I actualized sql/psm patch. This patch can be downloaded from
> > http://www.pgsql.cz/patches/plpgpsm.diff.gz
>
> The fundamental problem I've got with this patch is that it adds 400K
> of new code (and that's just the code, not counting documentation or
> regression tests) that we'll have to maintain, to obtain a feature
> that so far as I've heard there is precisely zero demand for.
That is likely because everyone knew he was working on it. Consider
this my +1 for pl/psm support.
>
> The duplicativeness of the code with plpgsql doesn't make this
> prospect any more pleasant, either.
However, I do agree with you here. I would much prefer it be cleaned up
into its own space.
>
> The idea would be a lot easier to swallow if the code were refactored
> to avoid the duplication with plpgsql.
+1.
Sincerely,
Joshua D. Drake
- --
The PostgreSQL Company since 1997: http://www.commandprompt.com/
PostgreSQL Community Conference: http://www.postgresqlconference.org/
United States PostgreSQL Association: http://www.postgresql.us/
Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
iD8DBQFH9EIYATb/zqfZUUQRAmkiAJ9Q5s2Lsit7lW60sczr1/wxEGX2LACdH2rl
079G3tiL/Jj+B7FU6G5e65c=
=nnba
-----END PGP SIGNATURE-----
From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | "Jonah H(dot) Harris" <jonah(dot)harris(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 04:34:54 |
Message-ID: | 47F45E6E.1050706@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Jonah H. Harris wrote:
> On Tue, Apr 1, 2008 at 5:55 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
>> The fundamental problem I've got with this patch is that it adds 400K
>> of new code (and that's just the code, not counting documentation or
>> regression tests) that we'll have to maintain, to obtain a feature that
>> so far as I've heard there is precisely zero demand for.
>>
>
> We have a customer that wants to use it as part of a MySQL-to-Postgres
> migration.
>
>
Using an implementation like this? I suspect anyone wanting to migrate
their existing SQL/PSM stuff to Postgres will be less than impressed by
our "function body as a string" mechanism.
cheers
andrew
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com> |
Cc: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 04:57:11 |
Message-ID: | 23134.1207198631@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
"Joshua D. Drake" <jd(at)commandprompt(dot)com> writes:
> Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> The fundamental problem I've got with this patch is that it adds 400K
>> of new code (and that's just the code, not counting documentation or
>> regression tests) that we'll have to maintain, to obtain a feature
>> that so far as I've heard there is precisely zero demand for.
> That is likely because everyone knew he was working on it.
By "everyone" I suppose you mean the dozen or three people who are
paying close attention to who's doing what in PG development. The
above argument is hogwash, really. If SQL/PSM support were so widely
desired as to justify a code addition of this size, then the archives
would be littered with requests for it. Try to find some. (As a
reasonable comparison point for what it takes to justify a large
code addition, compare that to the number of times that text search
requests show up --- most of them coming from people who don't know
who Oleg and Teodor are.)
I'm not against having SQL/PSM support. I'm just saying I'm not
willing to support two copies of plpgsql to do it.
regards, tom lane
From: | "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 05:02:29 |
Message-ID: | 20080402220229.79b10840@commandprompt.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Thu, 03 Apr 2008 00:57:11 -0400
Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> "Joshua D. Drake" <jd(at)commandprompt(dot)com> writes:
> > Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> >> The fundamental problem I've got with this patch is that it adds
> >> 400K of new code (and that's just the code, not counting
> >> documentation or regression tests) that we'll have to maintain, to
> >> obtain a feature that so far as I've heard there is precisely zero
> >> demand for.
>
> > That is likely because everyone knew he was working on it.
>
> By "everyone" I suppose you mean the dozen or three people who are
> paying close attention to who's doing what in PG development.
Well I think that is a bit of an understatement. I know that I have
talked to people about this patch for some time. Even well before 8.3
came out.
> I'm not against having SQL/PSM support. I'm just saying I'm not
> willing to support two copies of plpgsql to do it.
I didn't disagree with you Tom.
Joshua D. Drake
- --
The PostgreSQL Company since 1997: http://www.commandprompt.com/
PostgreSQL Community Conference: http://www.postgresqlconference.org/
United States PostgreSQL Association: http://www.postgresql.us/
Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
iD8DBQFH9GTnATb/zqfZUUQRAlaqAJ0bU/N625e5+BoVQRepETsU4Lij5gCfQ5qo
xOqTAATx8P9AW7ZKE0qAE+I=
=g2v9
-----END PGP SIGNATURE-----
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | "Jonah H(dot) Harris" <jonah(dot)harris(at)gmail(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 05:04:15 |
Message-ID: | 23275.1207199055@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> ... I suspect anyone wanting to migrate
> their existing SQL/PSM stuff to Postgres will be less than impressed by
> our "function body as a string" mechanism.
Yeah, that's the other little problem with claiming standards-compliance
as a reason for doing this. We'd really have to suck it up and figure
some other way of parsing function bodies.
regards, tom lane
From: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 06:09:31 |
Message-ID: | 162867790804022309p2f5d02fcsdb9e53a45fa0611a@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Hello
>
> I'm not against having SQL/PSM support. I'm just saying I'm not
> willing to support two copies of plpgsql to do it.
>
> regards, tom lane
>
I understand it well. Pending development of plpgpsm I respected
unbreakability plpgsql. So I can move duplicate parts to separate
files and I'll do it.
I thinking about new directory structure (some like)
pl/sqlsp/ .. sql Stored Procedures
pl/sqlsp/utils
pl/sqlsp/plpgsql - only plpgpsm code
pl/sqlsp/plpgpsm - only plpgsql code
Regards
Pavel Stehule
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | "Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 14:18:56 |
Message-ID: | 29194.1207232336@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
"Pavel Stehule" <pavel(dot)stehule(at)gmail(dot)com> writes:
> I thinking about new directory structure (some like)
> pl/sqlsp/ .. sql Stored Procedures
> pl/sqlsp/utils
> pl/sqlsp/plpgsql - only plpgpsm code
> pl/sqlsp/plpgpsm - only plpgsql code
Maybe "common" instead of "utils"?
Also, where did "plpgpsm" come from? This is supposed to be a standard,
so there is no reason to designate it as PG-specific. plpsm seems the
right name.
regards, tom lane
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | "Jonah H(dot) Harris" <jonah(dot)harris(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 15:20:52 |
Message-ID: | 200804031520.m33FKqB07912@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Andrew Dunstan wrote:
>
>
> Jonah H. Harris wrote:
> > On Tue, Apr 1, 2008 at 5:55 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> >
> >> The fundamental problem I've got with this patch is that it adds 400K
> >> of new code (and that's just the code, not counting documentation or
> >> regression tests) that we'll have to maintain, to obtain a feature that
> >> so far as I've heard there is precisely zero demand for.
> >>
> >
> > We have a customer that wants to use it as part of a MySQL-to-Postgres
> > migration.
> >
> >
>
> Using an implementation like this? I suspect anyone wanting to migrate
> their existing SQL/PSM stuff to Postgres will be less than impressed by
> our "function body as a string" mechanism.
What is your point? That because of the $$ strings they might as well
rewrite the whole thing in PL/pgSQL.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, "Jonah H(dot) Harris" <jonah(dot)harris(at)gmail(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: actualized SQL/PSM patch |
Date: | 2008-04-03 15:29:24 |
Message-ID: | 200804031529.m33FTOk11519@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-patches |
Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> > ... I suspect anyone wanting to migrate
> > their existing SQL/PSM stuff to Postgres will be less than impressed by
> > our "function body as a string" mechanism.
>
> Yeah, that's the other little problem with claiming standards-compliance
> as a reason for doing this. We'd really have to suck it up and figure
> some other way of parsing function bodies.
Oh, I understand now, that we aren't going to be 100% standards
compliant based on how we quote our function bodies --- I understand
Andrew's point now.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +