Re: Add some regression tests for SEQUENCE

From: Robins Tharakan <tharakan(at)gmail(dot)com>
To: Josh Berkus <josh(at)agliodbs(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
Subject: Re: Add some regression tests for SEQUENCE
Date: 2013-06-28 21:15:30
Message-ID: CAEP4nAwkRX+y9ioTfACbBC4tpk6M1T5kL_b7wuyZz_ronkZrhg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Seems like thats because of a recent (15th May 2013) patch
(b14206862278347a379f2bb72d92d16fb9dcea45) that changed the error message
that is printed. Its just one line of difference actually.

Let me know if this is otherwise good to go.
I'll checkout the latest revision and submit this patch again.

--
Robins Tharakan

On 28 June 2013 15:53, Josh Berkus <josh(at)agliodbs(dot)com> wrote:

> On 05/07/2013 03:40 PM, Robins Tharakan wrote:
> > Hi,
> >
> > Have provided an updated patch as per Fabien's recent response on
> > Commitfest site.
> > Any and all feedback is appreciated.
>
> The updated patch is giving a FAILURE for me:
>
> parallel group (19 tests): limit temp plancache conversion rowtypes
> prepare without_oid copy2 xml returning rangefuncs polymorphism with
> domain truncate largeobject sequence alter_table plpgsql
> plancache ... ok
> limit ... ok
> plpgsql ... ok
> copy2 ... ok
> temp ... ok
> domain ... ok
> rangefuncs ... ok
> prepare ... ok
> without_oid ... ok
> conversion ... ok
> truncate ... ok
> alter_table ... ok
> sequence ... FAILED
>
> Thoughts?
>
> --
> Josh Berkus
> PostgreSQL Experts Inc.
> http://pgexperts.com
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Josh Berkus 2013-06-28 21:22:15 Re: Add some regression tests for SEQUENCE
Previous Message Nicholas White 2013-06-28 21:07:45 Re: Request for Patch Feedback: Lag & Lead Window Functions Can Ignore Nulls