Need to know transaction.isReadonly while getting connection - spring-jdbc

[johnny2002]

Backgroup: I have a large project with more than 500 nodes need to connection to one Database, so connection count is too high. In order to reduce db connections, I want to reuse one connection for all readonly transactions. So, I need to get the "TransactionSynchronizationManager.isCurrentTransactionReadOnly()" in Datasource.getConnection() method. While, in spring-jdbc-5.2.2, DataSourceTransactionManager.doBegin method, the connection is gotten at line 263, the readonly flag is setted at line 298.

Suggestion: Set the readonly flag before getting connection. e.g. in DataSourceTransactionManager.doBegin method

protected void doBegin(Object transaction, TransactionDefinition definition) {
TransactionSynchronizationManager.setCurrentTransactionReadOnly(definition.isReadOnly());
DataSourceTransactionObject txObject = (DataSourceTransactionObject) transaction;
...
Connection newCon = obtainDataSource().getConnection();
...

Then I will override a Datasource, that, return one shared connection to all readonly transactions.

[quaff]

Is it safe to reuse connection across threads?

[johnny2002]

Is it safe to reuse connection across threads?

for readonly operations, it is.

[quaff]

Is it safe to reuse connection across threads?

for readonly operations, it is.

It depends on driver implementation.

[johnny2002]

Is it safe to reuse connection across threads?

for readonly operations, it is.

It depends on driver implementation.

At least, as I tested, oracle, mysql, pgsql are no problem.

[johnny2002]

further more, no matter share connection is safe or not, let developers can get the readonly property as earlier as possible is a good manner. Developers may use the readonly property for other purpos.

[johnny2002]

Is it safe to reuse connection across threads?

for readonly operations, it is.

It depends on driver implementation.

So, what's the decision? will you adopt the change request?

[snicoll]

The PR has been declined, reopening to reconsider this.