Unverified Commit c8b0b93e authored by zhaojun's avatar zhaojun Committed by GitHub
Browse files

Merge pull request #2217 from mfeng-ya/dev

use EnumMap replace HashMap for ShardingTransactionManagerEngine.
parents 7e7bd3bc 2208fe48
Loading
Loading
Loading
Loading
+13 −10
Original line number Diff line number Diff line
@@ -17,22 +17,25 @@

package org.apache.shardingsphere.transaction;

import com.google.common.base.Preconditions;
import lombok.extern.slf4j.Slf4j;
import org.apache.shardingsphere.core.constant.DatabaseType;
import org.apache.shardingsphere.transaction.core.ResourceDataSource;
import org.apache.shardingsphere.transaction.core.TransactionType;
import org.apache.shardingsphere.transaction.spi.ShardingTransactionManager;

import javax.sql.DataSource;
import java.util.Collection;
import java.util.HashMap;
import java.util.EnumMap;
import java.util.LinkedList;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;
import java.util.ServiceLoader;

import javax.sql.DataSource;

import org.apache.shardingsphere.core.constant.DatabaseType;
import org.apache.shardingsphere.transaction.core.ResourceDataSource;
import org.apache.shardingsphere.transaction.core.TransactionType;
import org.apache.shardingsphere.transaction.spi.ShardingTransactionManager;

import com.google.common.base.Preconditions;

import lombok.extern.slf4j.Slf4j;

/**
 * Sharding transaction manager engine.
 *
@@ -41,7 +44,7 @@ import java.util.ServiceLoader;
@Slf4j
public final class ShardingTransactionManagerEngine {
    
    private final Map<TransactionType, ShardingTransactionManager> transactionManagerMap = new HashMap<>();
    private final Map<TransactionType, ShardingTransactionManager> transactionManagerMap = new EnumMap<>(TransactionType.class);
    
    public ShardingTransactionManagerEngine() {
        loadShardingTransactionManager();