2017-07-18 8 views
0

{organizationId, userId} -> userEmailのキャッシュを保存したいのですが、利用可能なAPIは特定の組織のすべてのメールを返します。私がこれらの値をすべて取得している限り、CacheLoader::loadへの呼び出し中にそれらをすべて保存するのは安全ですか?CacheLoader :: load呼び出し中にGuavaキャッシュ内の無関係な値を更新することは安全ですか?

private final LoadingCache<Pair<UUID, UUID>, String> emailCache = CacheBuilder 
     .newBuilder() 
     .maximumSize(10000) 
     .build(new CacheLoader<Pair<UUID, UUID>, String>() { 
      @Override 
      public String load(final Pair<UUID, UUID> key) throws Exception { 
       final UUID orgId = key.getValue0(); 
       final List<User> users = remoteService.getAllUsers(orgId); 
       final Map<Pair<UUID, UUID>, String> updates = new HashMap<>(); 
       for (User user : users) { 
        updates.put(Pair.with(orgId, user.getId()), user.getEmail()); 
       } 

       // is this safe? 
       emailCache.putAll(updates); 

       return updates.get(key); 
      } 
     }); 
+0

代わりに、キーとしてorgIdを使用し、値として不変のマップを使用することもできます。次に、地図のサイズに基づいて秤を使用します。これにより、大量のアイテムは保持されますが、マップ全体が削除され、値の小さいマッピングでは余分なスペースが消費される可能性があります。 –

答えて

2

いいえ、レースを引き起こす可能性はありません。一方、CacheLoader.loadAllを使用すると安全です。具体的には、要求された数よりも多くのエントリを持つマップを返すことができることを文書化しています。

関連する問題