Re: [patch] plproxy v2 - Mailing list pgsql-hackers

On 7/8/08, Joshua D. Drake <jd@commandprompt.com> wrote:
>  On Tue, 2008-07-08 at 18:43 +0300, Marko Kreen wrote:
>  > On 7/8/08, Joshua D. Drake <jd@commandprompt.com> wrote:
>
> > > First let me say that I too enjoy PL/Proxy quite a bit. However, I don't
>  > >  think it needs to be in core. I wouldn't mind seeing it in contrib (or
>  > >  better yet modules/ should we ever get around to that).
>  >
>  > I'm not against contrib/ considering that the docs are now nicely
>  > integrated, but then, whats the difference between src/pl/ and contrib/?
>
> I am actually against adding to the grammar which is what Simon was
>  suggesting. If it wants to go into src/pl I wouldn't have a problem with
>  that. Sorry for not being more clear.

Current patch is for src/pl/.  Diffstat for patch v2:
doc/src/sgml/filelist.sgml                         |    1doc/src/sgml/plproxy.sgml                          |  221
++++++doc/src/sgml/postgres.sgml                        |    1src/include/catalog/pg_pltemplate.h                |
1src/pl/Makefile                                   |    2src/pl/plproxy/Makefile                            |   89
++src/pl/plproxy/cluster.c                          |  469 +++++++++++++src/pl/plproxy/execute.c
  |  724 +++++++++++++++++++++src/pl/plproxy/expected/plproxy_clustermap.out     |   71
++src/pl/plproxy/expected/plproxy_dynamic_record.out|   51 +src/pl/plproxy/expected/plproxy_errors.out         |   66
+src/pl/plproxy/expected/plproxy_init.out          |    2src/pl/plproxy/expected/plproxy_many.out           |  116
+++src/pl/plproxy/expected/plproxy_select.out        |   37 +src/pl/plproxy/expected/plproxy_test.out           |  312
+++++++++src/pl/plproxy/function.c                         |  479 +++++++++++++src/pl/plproxy/main.c
         |  214 ++++++src/pl/plproxy/parser.y                            |  203 +++++src/pl/plproxy/plproxy.h
               |  301 ++++++++src/pl/plproxy/poll_compat.c                       |  140
++++src/pl/plproxy/poll_compat.h                      |   58 +src/pl/plproxy/query.c                             |  316
+++++++++src/pl/plproxy/result.c                           |  222 ++++++src/pl/plproxy/rowstamp.h
  |   27src/pl/plproxy/scanner.l                           |  320 +++++++++src/pl/plproxy/sql/plproxy_clustermap.sql
     |   56 +src/pl/plproxy/sql/plproxy_dynamic_record.sql      |   43 +src/pl/plproxy/sql/plproxy_errors.sql
  |   63 +src/pl/plproxy/sql/plproxy_init.sql                |   57 +src/pl/plproxy/sql/plproxy_many.sql
|  66 +src/pl/plproxy/sql/plproxy_select.sql              |   37 +src/pl/plproxy/sql/plproxy_test.sql                |
200+++++src/pl/plproxy/type.c                              |  336 +++++++++33 files changed, 5299 insertions(+), 2
modifications(!)


-- 
marko