MySQL Replication Driver Support

[DigitalSmile]

Is there any way to get ReplicationDriver for MySQL (http://dev.mysql.com/doc/connectors/en/connector-j-master-slave-replication-connection.html) work out of the box with MyBatis?

I believe the main issue is in the configuration of DataSource and the statement executed. If the DataSource issue is clear for me (simply override method getConnection() to create new one and set the readOnly flag to true), the statement part is not. May be an option to the statement or environment configuration should be added to control which statements should be executed on slaves?

[emacarron]

It it is an statement configuration option then it can to the xml or to the @options parameter. (this will require a change in mybatis for sure, but looks easy).

But I wonder it this should be an statement property or a SqlSession method...

BTW, if you are using MyBatis with Spring, Spring sets the readOnly flag got from the Transaction to the Connection, so you may already have a solution.

[DigitalSmile]

I have had a discussion with my team and It seems the statement-based is the one we should stick to. Session-based could be tricky, because several statements (not only SELECT's) could be used inside.

We are using xml to provide MyBatis with SQL statements and, from our point of view, the most helpful would be parameter for select section, something like:

<select id="getEventIdsByUserRoles" parameterType="map" resultType="int" readOnly="true">

BTW, if you are using MyBatis with Spring, Spring sets the readOnly flag got from the Transaction to the Connection, so you may already have a solution.

We are not using Spring, so this is not our case...

Sorry for the late reply. Take care.

[DigitalSmile]

Just a quick think about this feature - It seems, if we are in transaction this flag should be ignored, because there could be any statements, not only SELECT's. Also statements inside transaction could rely on each other - in that case running SELECT on slave could return unpredicted results for other statements. Anyway, the transaction, consisting of several SELECT's is pointless...

[kazuki43zoo]

I've removed milestone.

@emacarron @harawata
Please set an appropriate milestone.