spring事务在readonly上的一个疏忽还是一个bug

问题描述：
http://www.iteye.com/topic/319768

经过测试加上源代码，发现， 如果你使用的是DataSourceTransactionManager来管理事务，readonly是不起作用的:（HibernateTransactionManager是起作用的，后面有源码分析）

Xml代码  

1. <bean id="transactionManager"  
2.     class="org.springframework.jdbc.datasource.DataSourceTransactionManager">  
3.     <property name="dataSource" ref="dataSource" />  
4. </bean>  
5.   
6. <tx:advice id="txAdvice" transaction-manager="transactionManager">  
7.     <tx:attributes>  
8.         <tx:method name="search*" read-only="true" />  
9.     </tx:attributes>  
10. </tx:advice>  

先看源码：
1. DataSourceTransactionManager.doBegain 方法：

Java代码  

1. protected void doBegin(Object transaction, TransactionDefinition definition) {  
2.         DataSourceTransactionObject txObject = (DataSourceTransactionObject) transaction;  
3.         Connection con = null;  
4.   
5.         try {  
6.             if (txObject.getConnectionHolder() == null ||  
7.                     txObject.getConnectionHolder().isSynchronizedWithTransaction()) {  
8.                 Connection newCon = this.dataSource.getConnection();  
9.                 if (logger.isDebugEnabled()) {  
10.                     logger.debug("Acquired Connection [" + newCon + "] for JDBC transaction");  
11.                 }  
12.                 txObject.setConnectionHolder(new ConnectionHolder(newCon), true);  
13.             }  
14.   
15.             txObject.getConnectionHolder().setSynchronizedWithTransaction(true);  
16.             con = txObject.getConnectionHolder().getConnection();  
17.   
18.             Integer previousIsolationLevel = DataSourceUtils.prepareConnectionForTransaction(con, definition);  
19.             txObject.setPreviousIsolationLevel(previousIsolationLevel);  
20.   
21.             // Switch to manual commit if necessary. This is very expensive in some JDBC drivers,  
22.             // so we don't want to do it unnecessarily (for example if we've explicitly  
23.             // configured the connection pool to set it already).  
24.             if (con.getAutoCommit()) {  
25.                 txObject.setMustRestoreAutoCommit(true);  
26.                 if (logger.isDebugEnabled()) {  
27.                     logger.debug("Switching JDBC Connection [" + con + "] to manual commit");  
28.                 }  
29.                 con.setAutoCommit(false);  
30.             }  
31.             txObject.getConnectionHolder().setTransactionActive(true);  
32.               
33.             int timeout = determineTimeout(definition);  
34.             if (timeout != TransactionDefinition.TIMEOUT_DEFAULT) {  
35.                 txObject.getConnectionHolder().setTimeoutInSeconds(timeout);  
36.             }  
37.   
38.             // Bind the session holder to the thread.  
39.             if (txObject.isNewConnectionHolder()) {  
40.                 TransactionSynchronizationManager.bindResource(getDataSource(), txObject.getConnectionHolder());  
41.             }  
42.         }  
43.   
44.         catch (SQLException ex) {  
45.             DataSourceUtils.releaseConnection(con, this.dataSource);  
46.             throw new CannotCreateTransactionException("Could not open JDBC Connection for transaction", ex);  
47.         }  
48.     }  

2. HibernateTransactionManager.doBegain方法中的一段：

Java代码  

1. protected void doBegin(Object transaction, TransactionDefinition definition)  
2.     {  
3. //省略一些代码  
4.  if(!definition.isReadOnly() && !txObject.isNewSession())  
5.         {  
6.             FlushMode flushMode = session.getFlushMode();  
7.             if(flushMode.lessThan(FlushMode.COMMIT))  
8.             {  
9.                 session.setFlushMode(FlushMode.AUTO);  
10.                 txObject.getSessionHolder().setPreviousFlushMode(flushMode);  
11.             }  
12.         }  
13. }  

可以看出， DataSourceTransactionManager.doBegain方法中没有一处来判断readonly的，所以不起作用，如果我加上：

Java代码  

1. protected void doBegin(Object transaction, TransactionDefinition definition) {  
2. //硬代码：con.setReadOnly(true)  
3. con.setReadOnly(true);  
4.   
5. int timeout = determineTimeout(definition);  
6.             if (timeout != TransactionDefinition.TIMEOUT_DEFAULT) {  
7.                 txObject.getConnectionHolder().setTimeoutInSeconds(timeout);  
8.             }  
9. }  

那么可以成功，看后台：

Xml代码  

1. nested exception is java.sql.SQLException: ORA-01456: may not perform insert/delete/update operation inside a READ ONLY transaction  
2.   
3.     at org.springframework.jdbc.support.SQLStateSQLExceptionTranslator.translate(SQLStateSQLExceptionTranslator.java:124)  

结论：如果你使用DataSourceTransactionManager来控制事务，即使你开启了readonly也是无效的。当然你可以自己修改源代码，加上readonly的判断

但为何spring不控制在DataSourceTransactionManager的readonly，而开启在HibernateTransactionManager的呢，这是个问题！