Re: pgsql: pgbench: Add a real expression syntax to \set

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

In response to

Responses

Browse pgsql-committers by date

  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