bad syntax in setval() statement [#183388]

it should be clear from the attached patch that there is a syntax error in the the setval() statement for the role ID sequence. marking as critical b/c postgres installations fail.

i'm also pretty sure this issue needs to be fixed in the D6 schema -- it doesn't look like it's addressed there.

Comment	File	Size	Author
	pg_bad_syntax.patch	757 bytes	hunmonk

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #2

Gábor Hojtsy

he/him

Hungarian

Hungary

CreditAttribution: Gábor Hojtsy commented 18 October 2007 at 14:46

It does not seem like this code is anywhere in D6. I honestly don't know how sequences are done in Drupal 6 with pgsql, so I don't know where to look, but system.install and block.install are not helpful. The block install file only creates the table now.

Comment #3

Gábor Hojtsy

he/him

Hungarian

Hungary

CreditAttribution: Gábor Hojtsy commented 9 November 2007 at 22:47

@hunmonk: what is your base of being pretty sure about this error being there? (Wee see people testing D6 on postgresql installations and it seems to work for them quite well).

Comment #4

hunmonk CreditAttribution: hunmonk commented 10 November 2007 at 21:32

Status:

Patch (to be ported)

» Fixed

drupal6=# select * from role;
 rid |        name        
-----+--------------------
   1 | anonymous user
   2 | authenticated user
(2 rows)

drupal6=# SELECT * from role_rid_seq ;
 sequence_name | last_value | increment_by |      max_value      | min_value | cache_value | log_cnt | is_cycled | is_called 
---------------+------------+--------------+---------------------+-----------+-------------+---------+-----------+-----------
 role_rid_seq  |          2 |            1 | 9223372036854775807 |         1 |           1 |      32 | f         | t
(1 row)

from a fresh drupal install. so it looks like you're right :)