[#QUARTZ-488] JobStoreCMT isTxIsolationLevelReadCommitted property sets TRANSACTION_READ_UNCOMMITTED

Quartz Scheduler

JobStoreCMT isTxIsolationLevelReadCommitted property sets TRANSACTION_READ_UNCOMMITTED

Created: 18/Jul/06 06:09 AM Updated: 01/May/07 09:02 PM

Component/s:

Job Stores

Affects Version/s:

1.6, 1.5.2

Fix Version/s:

1.6.1

Description

« Hide

JobStoreCMT.getNonManagedTXConnection():

            if (isTxIsolationLevelReadCommitted()) {
                conn.setTransactionIsolation(Connection.TRANSACTION_READ_UNCOMMITTED);
            }

Either the property name is wrong, it should be checking if it is false, or it should be setting to Connection.TRANSACTION_READ_COMMITTED

I don't know which was the original intent.

All

Comments

Change History

Sort Order:

[ Permlink | « Hide ]

James House - [16/Mar/07 11:43 PM ]

I'm pretty sure read committed was the original intent. It would really take some digging (a few years back) to be sure.

[ Permlink | « Hide ]

Henri Yandell - [01/May/07 09:02 PM ]

To the branch:

svn ci -m "Fixing the JobStoreCMT so that isTxIsolationLevelReadCommitted() sets the isolation to TRA
NSACTION_READ_COMMITTED and not TRANSACTION_READ_UNCOMMITTED as per ~~QUARTZ-488~~"
Sending build.xml (oops...accident)
Sending src/java/org/quartz/impl/jdbcjobstore/JobStoreCMT.java
Transmitting file data ..
Committed revision 699.

To trunk:

svn ci -m "Fixing the JobStoreCMT so that isTxIsolationLevelReadCommitted() sets the isolation to TRANSAC
TION_READ_COMMITTED and not TRANSACTION_READ_UNCOMMITTED as per ~~QUARTZ-488~~" src/
Sending src/java/org/quartz/impl/jdbcjobstore/JobStoreCMT.java
Transmitting file data .
Committed revision 700.