"putIfAbsent"はCHMでどのように機能しますか？

Question

私はCassandraと協力し、Datastax Javaドライバを使用しています。準備した文をキャッシュして再利用しようとしています。

private static final ConcurrentHashMap<String, PreparedStatement> holder = new ConcurrentHashMap<>(); 

    public BoundStatement getStatement(String cql) { 
    Session session = TestUtils.getInstance().getSession(); 
    PreparedStatement ps = holder.get(cql); 
    // no statement is cached, create one and cache it now. 
    if (ps == null) { 
     holder.putIfAbsent(cql, session.prepare(cql)); 
    } 
    return ps.bind(); 
    } 

Prepared StatementおよびBoundStatementのdatastax java driverです。

このgetStatementメソッドは複数のスレッドで呼び出されるため、スレッドセーフであることを確認する必要があります。私はJava 7で作業しています。

2つの同じcqlプリペアドステートメントを取得すると、putIfAbsentはどうなりますか？コードスレッドは安全で、競合状態はありませんか？

アップデート： -

public BoundStatement getStatement(String cql) { 
    Session session = TestUtils.getInstance().getSession(); 
    PreparedStatement ps = holder.get(cql); 
    // no statement is cached, create one and cache it now. 
    if (ps == null) { 
     synchronized (this) { 
     ps = holder.get(cql); 
     if (ps == null) { 
      ps = session.prepare(cql); 
      holder.put(cql, ps); 
     } 
     } 
    } 
    return ps.bind(); 
    } 

Answer 1

あなたのコードは、任意のcqlパラメータに二回（またはそれ以上）と呼ばれてsession.prepare(cql)につながることができます競合状態を持っています。この場合、putIfAbsentは通常のputよりも利点がありません。

は、Java 8の上にいた場合、あなたは

PreparedStatement ps = holder.computeIfAbsent(cql, key -> session.prepare(key)); 

Answer 2

との重複を作成することなく、効率的にこれを書くことができますが、実際に競合状態を持っていますが、それは続けるとして、あなたのputIfAbsentは、おそらくまだ少し良いput純粋より古いステートメントなので、実際には2つのインスタンスが存在することはありません。競争状態の場合にのみ現れるので、利点は小さい。

Double-checked lockingのようです。本当に必要な場合は、再初期化の前後に同期ブロックを置くだけです。

PreparedStatementが変更可能であるため、私はキャッシュに懐疑的です。同じcqlに複数のインスタンスが必要な場合があります。これも実行可能ですが、適切に取得してリリースする必要があります。