opération de carte thread-safe

Je suis tombé sur le morceau de code suivant et j'ai noté quelques incohérences - pour un code multi-thread sécurisé.

    Map<String,Map<String,Set<String>> clusters = new HashMap<.........>;
    Map<String,Set<String>> servers = clusters.get(clusterkey);
    if(servers==null){
      synchronized(clusterkey){
       servers = clusters.get(clusterkey);
       if(servers==null){....initialize new hashmap and put...}
      }
    }
    Set<String> users=servers.get(serverkey);
    if(users==null){
      synchronized(serverkey){
       users=servers.get(serverkey);
       if(users==null){ ... initialize new hashset and put...}
      }
    }
    users.add(userid);

1. Pourquoi la carte serait-elle synchronisée sur la clé de cluster - ne devrait-elle pas être sur la carte en tant que moniteur lui-même?
2. Les derniers utilisateurs.add ... ne devraient-ils pas également être synchronisés?
3. Cela semble être beaucoup de code pour ajouter un seul utilisateur de manière thread-safe. Quelle serait une implémentation plus intelligente?