From: | "Marko Kreen" <markokr(at)gmail(dot)com> |
---|---|
To: | Cédric Villemain <cedric(dot)villemain(at)dalibo(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Proposal: include PL/Proxy into core |
Date: | 2007-03-30 11:06:36 |
Message-ID: | e51f66da0703300406p50ed57d3te29f7d9ff89a0303@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 3/30/07, Cédric Villemain <cedric(dot)villemain(at)dalibo(dot)com> wrote:
> Le vendredi 30 mars 2007 12:36, Marko Kreen a écrit:
> > Patch:
> >
> > http://plproxy.projects.postgresql.org/plproxy_core.diff.gz
> Note a perhaps oversight in your makefile :
>
> + #REGRESS_OPTS
> = --dbname=$(PL_TESTDB) --load-language=plpgsql --load-language=plproxy
> + REGRESS_OPTS
> = --dbname=regression --load-language=plpgsql --load-language=plproxy
Heh. The problem is I had 'regression' hardwired into
regtests, so I could not use $(PL_TESTDB).
If the proposal is accespted and we want to always run
PL/Proxy regtests, there should be some dynamic way
of passing main dbname and also connstrings for partitions
into regression tests.
ATM I thought it can stay as-is. (Actually I forgot that change
after I had done it :)
--
marko
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2007-03-30 11:21:29 | Re: [PATCHES] Full page writes improvement, code update |
Previous Message | Cédric Villemain | 2007-03-30 10:47:23 | Re: Proposal: include PL/Proxy into core |