setTransactionIsolation on JDBC Connection only called when actually necessary (for PostgreSQL; SPR-7184)

This commit is contained in:
Juergen Hoeller 2010-05-11 13:14:53 +00:00
parent 15fe75b1a1
commit 853eab8b4c
1 changed files with 7 additions and 5 deletions

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2008 the original author or authors. * Copyright 2002-2010 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -19,7 +19,6 @@ package org.springframework.jdbc.datasource;
import java.sql.Connection; import java.sql.Connection;
import java.sql.SQLException; import java.sql.SQLException;
import java.sql.Statement; import java.sql.Statement;
import javax.sql.DataSource; import javax.sql.DataSource;
import org.apache.commons.logging.Log; import org.apache.commons.logging.Log;
@ -170,9 +169,12 @@ public abstract class DataSourceUtils {
logger.debug("Changing isolation level of JDBC Connection [" + con + "] to " + logger.debug("Changing isolation level of JDBC Connection [" + con + "] to " +
definition.getIsolationLevel()); definition.getIsolationLevel());
} }
previousIsolationLevel = con.getTransactionIsolation(); int currentIsolation = con.getTransactionIsolation();
if (currentIsolation != definition.getIsolationLevel()) {
previousIsolationLevel = currentIsolation;
con.setTransactionIsolation(definition.getIsolationLevel()); con.setTransactionIsolation(definition.getIsolationLevel());
} }
}
return previousIsolationLevel; return previousIsolationLevel;
} }
@ -193,7 +195,7 @@ public abstract class DataSourceUtils {
logger.debug("Resetting isolation level of JDBC Connection [" + logger.debug("Resetting isolation level of JDBC Connection [" +
con + "] to " + previousIsolationLevel); con + "] to " + previousIsolationLevel);
} }
con.setTransactionIsolation(previousIsolationLevel.intValue()); con.setTransactionIsolation(previousIsolationLevel);
} }
// Reset read-only flag. // Reset read-only flag.