Issue Details (XML | Word | Printable)

Key: CONTINUUM-2147
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Maria Catherine Tan
Reporter: Maria Catherine Tan
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Continuum

Schema generated is incorrect when jpox.null-value="default" is set in sendOnScmFailure field

Created: 23/Mar/09 07:20 PM   Updated: 27/May/09 10:53 PM   Resolved: 23/Mar/09 07:24 PM
Return to search
Component/s: Database
Affects Version/s: 1.3.2
Fix Version/s: 1.3.3

Time Tracking:
Not Specified

Complexity: Intermediate


 Description  « Hide

jpox.null-value="default" in a boolean field causes:

1.) With MySQL, you won't be able to create and upgrade your database from Continuum 1.2.x to 1.3.x

ERROR JPOX.RDBMS - Error thrown executing CREATE TABLE `PROJECTNOTIFIER`
(
`ID` INTEGER NOT NULL,
`ENABLED` BIT NOT NULL,
`NOTIFIERORIGIN` INTEGER NOT NULL,
`MODEL_ENCODING` VARCHAR(255) BINARY NULL,
`RECIPIENT_TYPE` INTEGER NOT NULL,
`SEND_ON_ERROR` BIT NOT NULL,
`SEND_ON_FAILURE` BIT NOT NULL,
`SEND_ON_SCM_FAILURE` BIT NOT NULL DEFAULT 'false',
`SEND_ON_SUCCESS` BIT NOT NULL,
`SEND_ON_WARNING` BIT NOT NULL,
`NOTIFIERTYPE` VARCHAR(255) BINARY NULL,
PRIMARY KEY (`ID`)
) ENGINE=INNODB : Invalid default value for 'SEND_ON_SCM_FAILURE'
com.mysql.jdbc.exceptions.MySQLSyntaxErrorException: Invalid default value for 'SEND_ON_SCM_FAILURE'

2.) With Derby, you'll be able to create your database but it fails when you try to upgrade from continuum 1.2.x to 1.3.x

ERROR JPOX.RDBMS - Error thrown executing ALTER TABLE PROJECTNOTIFIER ADD SEND_ON_SCM_FAILURE CHAR(1) DEFAULT 'false' CHECK (SEND_ON_SCM_FAILURE IN ('Y','N')) : A truncation error was encountered trying to shrink CHAR 'false' to length 1.
ERROR 22001: A truncation error was encountered trying to shrink CHAR 'false' to length 1.



Maria Catherine Tan added a comment - 23/Mar/09 07:24 PM

Removed jpox.null-value="default" in sendOnScmFailure field - revision 757136


Brett Porter added a comment - 27/May/09 10:53 PM

note that some additional changes were made in r766106 to improve upgradeability on both databases