From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: pgbench: Add a real expression syntax to \set |
Date: | 2015-03-02 22:02:38 |
Message-ID: | CA+TgmobvrcD0UsxkhNooVYZZdOvKQgeCJJb+NjNEfyfbvPE1qA@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On Mon, Mar 2, 2015 at 2:38 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <rhaas(at)postgresql(dot)org> writes:
>> pgbench: Add a real expression syntax to \set
>
> The makefile changes in this do not look right to me. Files that are
> meant to be shipped in the tarball should be removed by make
> maintainer-clean, not an ordinary "make clean" which is what the committed
> patch appears to do. Otherwise, a tarball user without bison installed
> would be cut off at the knees if he did "make clean" or "make distclean";
> but those operations are not supposed to remove files that were in the
> tarball.
Hmm, yeah. Also, it appears that the exprscan.h file doesn't actually
get (or need to be) generated here. So I think we need something like
the attached.
Thought?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment | Content-Type | Size |
---|---|---|
pgbench-expr-build-fix.patch | binary/octet-stream | 957 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2015-03-02 22:17:54 | Re: pgsql: pgbench: Add a real expression syntax to \set |
Previous Message | Fabien COELHO | 2015-03-02 19:56:04 | Re: pgsql: pgbench: Add a real expression syntax to \set |