Add new option for force rollback on session close

Fixes gh-2363

Author: kazuki43zoo

src/main/java/org/apache/ibatis/builder/xml/XMLConfigBuilder.java

@@ -270,6 +270,7 @@ private void settingsElement(Properties props) {
     configuration.setConfigurationFactory(resolveClass(props.getProperty("configurationFactory")));
     configuration.setShrinkWhitespacesInSql(booleanValueOf(props.getProperty("shrinkWhitespacesInSql"), false));
     configuration.setDefaultSqlProviderType(resolveClass(props.getProperty("defaultSqlProviderType")));
+    configuration.setForceRollbackOnClose(booleanValueOf(props.getProperty("forceRollbackOnClose"), false));
   }
 
   private void environmentsElement(XNode context) throws Exception {

src/main/java/org/apache/ibatis/session/Configuration.java

@@ -114,6 +114,7 @@ public class Configuration {
   protected boolean useActualParamName = true;
   protected boolean returnInstanceForEmptyRow;
   protected boolean shrinkWhitespacesInSql;
+  protected boolean forceRollbackOnClose;
 
   protected String logPrefix;
   protected Class<? extends Log> logImpl;
@@ -297,6 +298,26 @@ public void setShrinkWhitespacesInSql(boolean shrinkWhitespacesInSql) {
     this.shrinkWhitespacesInSql = shrinkWhitespacesInSql;
   }
 
+  /**
+   * Whether force rollback on session close.
+   *
+   * @return if return {@code true}, execute force rollback
+   * @since 3.5.8
+   */
+  public boolean isForceRollbackOnClose() {
+    return forceRollbackOnClose;
+  }
+
+  /**
+   * Set whether force rollback on session close.
+   *
+   * @param forceRollbackOnClose if set {@code true}, execute force rollback
+   * @since 3.5.8
+   */
+  public void setForceRollbackOnClose(boolean forceRollbackOnClose) {
+    this.forceRollbackOnClose = forceRollbackOnClose;
+  }
+
   public String getDatabaseId() {
     return databaseId;
   }

src/main/java/org/apache/ibatis/session/defaults/DefaultSqlSession.java

@@ -257,7 +257,7 @@ public List<BatchResult> flushStatements() {
   @Override
   public void close() {
     try {
-      executor.close(isCommitOrRollbackRequired(false));
+      executor.close(isCommitOrRollbackRequired(configuration.isForceRollbackOnClose()));
       closeCursors();
       dirty = false;
     } finally {

src/test/java/org/apache/ibatis/builder/CustomizedSettingsMapperConfig.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <!--
 
-       Copyright 2009-2020 the original author or authors.
+       Copyright 2009-2021 the original author or authors.
 
        Licensed under the Apache License, Version 2.0 (the "License");
        you may not use this file except in compliance with the License.
@@ -56,6 +56,7 @@
     <setting name="defaultEnumTypeHandler" value="org.apache.ibatis.type.EnumOrdinalTypeHandler"/>
     <setting name="shrinkWhitespacesInSql" value="true"/>
     <setting name="defaultSqlProviderType" value="org.apache.ibatis.builder.XmlConfigBuilderTest$MySqlProvider"/>
+    <setting name="forceRollbackOnClose" value="true"/>
   </settings>
 
   <typeAliases>

src/test/java/org/apache/ibatis/builder/XmlConfigBuilderTest.java

@@ -103,6 +103,7 @@ void shouldSuccessfullyLoadMinimalXMLConfigFile() throws Exception {
       assertThat(config.getTypeHandlerRegistry().getTypeHandler(RoundingMode.class)).isInstanceOf(EnumTypeHandler.class);
       assertThat(config.isShrinkWhitespacesInSql()).isFalse();
       assertThat(config.getDefaultSqlProviderType()).isNull();
+      assertThat(config.isForceRollbackOnClose()).isFalse();
     }
   }
 
@@ -200,6 +201,7 @@ void shouldSuccessfullyLoadXMLConfigFile() throws Exception {
       assertThat(config.getConfigurationFactory().getName()).isEqualTo(String.class.getName());
       assertThat(config.isShrinkWhitespacesInSql()).isTrue();
       assertThat(config.getDefaultSqlProviderType().getName()).isEqualTo(MySqlProvider.class.getName());
+      assertThat(config.isForceRollbackOnClose()).isTrue();
 
       assertThat(config.getTypeAliasRegistry().getTypeAliases().get("blogauthor")).isEqualTo(Author.class);
       assertThat(config.getTypeAliasRegistry().getTypeAliases().get("blog")).isEqualTo(Blog.class);