The first line already has a default value

protected ExecutorType defaultExecutorType = ExecutorType.SIMPLE;

Author: ryh

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

@@ -698,7 +698,7 @@ public Executor newExecutor(Transaction transaction) {
 
   public Executor newExecutor(Transaction transaction, ExecutorType executorType) {
     executorType = executorType == null ? defaultExecutorType : executorType;
-    executorType = executorType == null ? ExecutorType.SIMPLE : executorType;
+//    executorType = executorType == null ? ExecutorType.SIMPLE : executorType;
     Executor executor;
     if (ExecutorType.BATCH == executorType) {
       executor = new BatchExecutor(this, transaction);