Re: CREATE TABLESPACE WITH - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: CREATE TABLESPACE WITH
Date
Msg-id CAB7nPqRpWyuiVnbwcLh+CXUt=cev-Yn8DJ8y=_67pzv0X6nGvg@mail.gmail.com
Whole thread Raw
In response to Re: CREATE TABLESPACE WITH  (Vik Fearing <vik.fearing@dalibo.com>)
Responses Re: CREATE TABLESPACE WITH  (Vik Fearing <vik.fearing@dalibo.com>)
List pgsql-hackers
On Thu, Jan 16, 2014 at 9:03 AM, Vik Fearing <vik.fearing@dalibo.com> wrote:
> New patch attached, with regression tests.

Thanks for the new version, I have spent some time looking at it:
- Patch compiles without warnings.
- Done some manual testing with CREATE/ALTER TABLESPACE WITH and
checked pg_tablespace, it worked fine.
- However, regression tests are failing because
src/test/regress/output/tablespace.source has an incorrect output, it
is necessary to replace that:
/home/vik/postgresql/9.4/git/src/test/regress/testtablespace
by that:
@testtablespace@
This is something I have actually fixed that in the attached patch.

So, except for the regression output problem, I think that the code is
clean so marking it as "Ready for committer" on the commit fest app.
Thanks,
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: currawong is not a happy animal
Next
From: Florian Pflug
Date:
Subject: Re: [PATCH] Negative Transition Aggregate Functions (WIP)