[Dbix-class] PK::Auto::Pg bugfix: GUINEA PIGS WANTED

[Krzysztof Krzyzaniak]

Brandon Black wrote:
> On 11/24/05, Krzysztof Krzyzaniak <eloy at kofeina.net> wrote:
> 
>>Matt S Trout wrote:
>>
>>>On Wed, Nov 23, 2005 at 01:31:32PM -0600, Brandon Black wrote:
>>>
>>>
>>>>Been away from the keyboard for a number of days, I'll get caught up
>>>>on the backlogged email after this weekend probably.  But in the
>>>>meantime...
>>>>
>>>>As of PostgreSQL 8.1, autoincrementing sequences can (and should) be
>>>>specified differently than before.  To cover the new syntax, a small
>>>>change needs to be made to Class/DBIx/PK/Auto/Pg.pm.  Works For Me,
>>>>YMMV:
>>>>
>>>>30c30
>>>><         m!^nextval\('"?([^"']+)"?'::(?:text|regclass)\)!;
>>>>---
>>>>
>>>>
>>>>>       m!^nextval\('"?([^"']+)"?'::text\)!;
>>>
>>>
>>>I (as usual) don't have a postgres install handy, can those who do apply and
>>>report results please?
>>
>>Dump in pg8.1 shows serial primary keys as:
>>
>>id bigint DEFAULT nextval(('account_id_seq'::text)::regclass) NOT NULL,
>>
>>I can test patch tonight.
>>
> 
> 
> That one-line patch definitely won't work for the case you're showing.
>  Was that column definition imported via pgdump from pre-8.1?  My

It's probably. It's devel environment which I've started with 8.0 and 
now is 8.1.

> serials created under 8.1 show up with "DEFAULT
> nextval('account_id_seq'::regclass)".  Here's the 8.1 ChangeLog entry
> that goes into detail about the change:
>

Right. So my case is probably not usual. I will run fixseq.sql and will 
give report.

-- 
-------e-l-o-y-----------e-l-o-y- at -t-r-a-n-s-i-l-v-a-n-i-a-.-e-u-.-o-r-g------

        jak to dobrze, że są oceany - bez nich byłoby jeszcze smutniej