Apache Roller
  1. Apache Roller
  2. ROL-910

Errors in PostgreSQL upgrade script and code for Roller 2.0

    Details

    • Database:
      PostgreSQL

      Description

      PostgreSQL users upgrading from Roller 1.3 to Roller 2.0 run into fatal errors during Roller startup.

      There are two parts to the problem:

      • The first query in upgradeTo200() is not compatible with PostgreSQL
      • The 130-to-200 script for PostgreSQL has multiple errors in the ALTER TABLE statements

      I've got a fix for this. At a mininum, we'll need to provide fixes for these files:

      • WEB-INF/lib/rollerbeans.jar
      • WEB-INF/dbscripts/postgresql/130-to-200-migration.sql

        Activity

        Hide
        David Johnson added a comment -

        Fixed alter table syntax that was causing problems with PostgreSQL.

        Did not fix index creation errors, they are harmless.

        Show
        David Johnson added a comment - Fixed alter table syntax that was causing problems with PostgreSQL. Did not fix index creation errors, they are harmless.
        Hide
        Douglas Robertson added a comment -

        The new script contains a new error in the following group of commands:

        alter table website add column emailaddress varchar(255);
        alter table website alter emailaddress set default '';
        update website set emailaddress='';
        alter table website alter handle set not null;

        The last SQL command should actually be:
        alter table website alter EMAILADDRESS set not null;

        (Note: the column name listed above should be lowercase, but has been uppered so as to point out the error)

        Show
        Douglas Robertson added a comment - The new script contains a new error in the following group of commands: alter table website add column emailaddress varchar(255); alter table website alter emailaddress set default ''; update website set emailaddress=''; alter table website alter handle set not null; The last SQL command should actually be: alter table website alter EMAILADDRESS set not null; (Note: the column name listed above should be lowercase, but has been uppered so as to point out the error)
        Hide
        David Johnson added a comment -

        Douglas, I just refreshed the Roller 2.0.1 download with a fix for this problem.

        Show
        David Johnson added a comment - Douglas, I just refreshed the Roller 2.0.1 download with a fix for this problem.
        Hide
        linda skrocki added a comment -

        Closing: old issue that was addressed w/ 2.0.1

        Show
        linda skrocki added a comment - Closing: old issue that was addressed w/ 2.0.1

          People

          • Assignee:
            David Johnson
            Reporter:
            David Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development