openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Patrick Linskey" <plins...@gmail.com>
Subject Re: comments about DataCacheStoreManager.transformToVersionSafePCDatas()
Date Fri, 04 Apr 2008 06:56:22 GMT
I added OPENJPA-556 to track this. I just submitted a change to the
queue; assuming it makes it through the regression tests, the next
snapshot build should have the change.

-Patrick

On Thu, Apr 3, 2008 at 10:48 AM,  <Frederic_Bellier@capgroup.com> wrote:
>
>
>  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
>
>



-- 
Patrick Linskey
202 669 5907

Mime
View raw message