연결 풀

Question

BlockingQueue<Connection> connections = new LinkedBlockingQueue<Connection>(maxConnection); 

AtomicInteger numberOfDrewedConnectionFromPool 

내 ConnectionPool에서 나는 LinkedBlockingQueue를 사용합니다. 곱하기 "if 문"이 스레드로부터 안전할지 여부에 대해서는 의심의 여지가 있습니다. maxConnection은 상수입니다. numberOfDrewedConnectionFromPool 잠금없이 방법 releaseConection도 변경() ..

public Connection getConnection() throws ConnectionPoolException { 
    Connection connection = null; 

    if ((connections.poll() == null) && (maxConnection > numberOfDrewedConnectionFromPool.get())) { 
     return newConnection(); 
    } else { 
     return connections.poll(); 
    } 
} 


private Connection newConnection() throws ConnectionPoolException { 
    lock.lock(); 
    Connection connection = null; 
    try { 
     try { 
      connection = DriverManager.getConnection(url, user, password); 
      numberOfDrewedConnectionFromPool.incrementAndGet(); 
     } catch (SQLException exception) { 
      throw new ConnectionPoolException(); 
     } 
    } finally { 
     lock.unlock(); 
     return connection; 
    } 

} 

Answer 1

아니, 코드가 스레드로부터 안전하지 않습니다.

connections.poll()이 호출 된 시간과 연결 수를 max와 비교하는 시간 사이에 다른 스레드가 연결을 해제했거나 연결했을 수 있으며 연결 수가 변경되었을 수 있습니다. 또한 대기열을 두 번 폴링하여 단일 연결을 얻습니다.

사이드 노트 : 왜 바퀴를 다시 발명하고 있습니까? 가능한 많은 무료 연결 풀이 있습니다.