openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Frederic_Bell...@capgroup.com
Subject comments about DataCacheStoreManager.transformToVersionSafePCDatas()
Date Thu, 03 Apr 2008 17:48:43 GMT


The DataCacheStoreManager.transformToVersionSafePCDatas()  line 261. This
method should call either cache.containsAll() or cache.getAll(). The
current implementation makes one call to the cache for each element in the
collection.


The AbstractDataCache.putAllInternal() method suffers from the same issues.
This is OK becuase it can be overwritten in a plugin. But I am thinking
that maybe someone from the openjpa developer community might want to fix
that in there too. That way the base class could actually be used as a base
class to write other L2 cache plugins.

F


Mime
View raw message