Bugs item #1042377, was opened at 2004-10-07 11:30
Message generated for change (Comment added) made by skrjboss
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=376685&aid=1042377&group_id=22866
Category: JBossServer
Group: v4.0
Status: Open
Resolution: None
Priority: 5
Submitted By: bossman (skrjboss)
Assigned to: Nobody/Anonymous (nobody)
Summary: setJdbcTransactionIsolation() no worky. Fix included.
Initial Comment:
package: org.jboss.resource.adapter.jdbc;
Class: BaseWrapperManagedConnection
BaseWrapperManagedConnection.setJdbcTransactionIsol
ation() method is not using the variable passed in to set
the isolation level.
Here is the current code
void setJdbcTransactionIsolation(final int isolationLevel)
throws SQLException
{
synchronized (stateLock)
{
con.setTransactionIsolation
(jdbcTransactionIsolation); <-------- HERE IS THE BUG.
Should use var isolationLevel instead of
jdbcTransactionIsolation.
this.jdbcTransactionIsolation = isolationLevel;
}
}
----------------------------------------------------------------------
>Comment By: bossman (skrjboss)
Date: 2004-10-07 12:17
Message:
Logged In: YES
user_id=1134885
Also, in the cleanup() method of the same class, when
resetting the isolation level, the code should use the "default"
one which is this.transactionIsolation.
In cleanup method...
synchronized (stateLock)
{
jdbcAutoCommit = true;
jdbcReadOnly = readOnly;
if (jdbcTransactionIsolation != transactionIsolation)
{
try
{
con.setTransactionIsolation
(jdbcTransactionIsolation); <- BUG. This should be
transactionIsolation.
jdbcTransactionIsolation = transactionIsolation;
}
catch (SQLException e)
{
mcf.log.warn("Error resetting transaction
isolation ", e);
}
}
}
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=376685&aid=1042377&group_id=22866
-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
JBoss-Development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development
