castor

Can't use Database.load(...) when all the columns in the table are part of the primary key

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.2
  • Fix Version/s: 1.3 rc1
  • Component/s: JDO
  • Labels:
    None
  • Environment:
    Windows, jtds-1.2.2.jar, MSSQL 2005, JDK is 1.4.2_08 from SUN
  • Testcase included:
    yes
  • Number of attachments :
    2

Description

I'm getting this error when use the Database.load(...) when my table has only columns wich are part of the primary key:

java.lang.IllegalStateException: Transaction org.castor.persist.LocalTransactionContext@147c1db does not hold the read lock
...
java.lang.ArrayIndexOutOfBoundsException: 0
at org.exolab.castor.jdo.engine.SQLStatementLoad.executeStatement(SQLStatementLoad.java:332)
at org.exolab.castor.jdo.engine.SQLEngine.load(SQLEngine.java:349)
...

The table the unitTest:

create table FSIM17(
NUMCIA decimal(3,0) NOT NULL,
NUMALM decimal(3,0) NOT NULL,
IPWAS CHAR(15) NOT NULL,
primary key (NUMCIA,NUMALM,IPWAS)
)

  1. File
    castorTest.7z
    09/Apr/08 5:37 PM
    18 kB
    Victor Manuel Iniestra Alvarez
  2. Text File
    patch.c2326.20080411.txt
    11/Apr/08 4:39 AM
    0.7 kB
    Werner Guttmann

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. CASTOR-2133 ArayIndexOutOfBoundException at SQLStatementLoad at line 344

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Werner Guttmann made changes -
Field Original Value New Value
Assignee Werner Guttmann [ wguttmn ]
Werner Guttmann made changes -
Original Estimate 1 hour, 30 minutes [ 5400 ]
Remaining Estimate 1 hour, 30 minutes [ 5400 ]
Fix Version/s 1.2.1 [ 14004 ]
Hide
Permalink
Werner Guttmann added a comment -

Initial patch for review.

Show
Werner Guttmann added a comment - Initial patch for review.
Werner Guttmann made changes -
Attachment patch.c2326.20080411.txt [ 33799 ]
Permalink
Werner Guttmann logged work - 11/Apr/08 4:40 AM
  • Time Spent:
    20 minutes
     
    Patch creation & integration testing.
Werner Guttmann made changes -
Remaining Estimate 1 hour, 30 minutes [ 5400 ] 1 hour, 10 minutes [ 4200 ]
Time Spent 20 minutes [ 1200 ]
Werner Guttmann made changes -
Remaining Estimate 1 hour, 10 minutes [ 4200 ] 50 minutes [ 3000 ]
Time Spent 20 minutes [ 1200 ] 40 minutes [ 2400 ]
Hide
Permalink
Werner Guttmann added a comment -

As can be seen in the patch easily, an additional null check resolved the problem. I still have to add anew test case so that we are not going to be hit by this problem again.

Show
Werner Guttmann added a comment - As can be seen in the patch easily, an additional null check resolved the problem. I still have to add anew test case so that we are not going to be hit by this problem again.
Permalink
Werner Guttmann logged work - 11/Apr/08 6:17 AM
  • Time Spent:
    45 minutes
     
    Added new test case to CPA CTF.
Werner Guttmann made changes -
Remaining Estimate 50 minutes [ 3000 ] 5 minutes [ 300 ]
Time Spent 40 minutes [ 2400 ] 1 hour, 25 minutes [ 5100 ]
Werner Guttmann made changes -
Resolution Fixed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]
Hide
Permalink
Victor Manuel Iniestra Alvarez added a comment -

It works on my enviroment. Thanks.

Show
Victor Manuel Iniestra Alvarez added a comment - It works on my enviroment. Thanks.
Hide
Permalink
Werner Guttmann added a comment -

Does that imply that you built the binaries yourself from SVN trunk after I have committed the patch ?

Show
Werner Guttmann added a comment - Does that imply that you built the binaries yourself from SVN trunk after I have committed the patch ?
Hide
Permalink
Victor Manuel Iniestra Alvarez added a comment -

Yes, it does. I downloaded the sources a week ago.

Show
Victor Manuel Iniestra Alvarez added a comment - Yes, it does. I downloaded the sources a week ago.
Ralf Joachim made changes -
Link This issue is duplicated by CASTOR-2133 [ CASTOR-2133 ]
Hide
Permalink
Werner Guttmann added a comment -

In that case, you might want to close this issue to let us know you have tested our resolution and are happy about this.

Show
Werner Guttmann added a comment - In that case, you might want to close this issue to let us know you have tested our resolution and are happy about this.
Werner Guttmann made changes -
Fix Version/s 1.3 [ 14004 ]
Fix Version/s 1.3 rc1 [ 14613 ]
Ralf Joachim made changes -
Status Resolved [ 5 ] Closed [ 6 ]

People

  • Assignee:
    Werner Guttmann
    Reporter:
    Victor Manuel Iniestra Alvarez
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

Estimated:
1h 30m
Original Estimate - 1 hour, 30 minutes
Remaining:
5m
Time Spent - 1 hour, 25 minutes Remaining Estimate - 5 minutes
Logged:
1h 25m
Time Spent - 1 hour, 25 minutes Remaining Estimate - 5 minutes