You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by Donald Woods <dw...@apache.org> on 2010/03/19 03:49:34 UTC
Re: svn commit: r924793 - in /geronimo/specs/trunk: geronimo-ejb_3.1_spec/
geronimo-el_2.2_spec/ geronimo-jacc_1.1_spec/ geronimo-jaspic_1.0_spec/ geronimo-jaxr_1.0_spec/
geronimo-jaxrpc_1.1_spec/ geronimo-jpa_2.0_spec/src/main/java/javax/persistence/
gero...
Rick, one comment on the geronimo-jpa_2.0_spec changes -
I'm adding the following back in, as the JPA 2.0 Spec suggests using a
provider cache per classloader and even requires implementing a
public void clearCachedProviders()
method to handle flushing the cache.
Otherwise, initial quick scan of changes (w/o testing) looks good.
I'll run the stand-alone JPA 2.0 TCK next week, after we get our OpenJPA
2.0.0 branch created.
-Donald
On 3/18/10 10:14 AM, rickmcguire@apache.org wrote:
> -
> - // cache the discovered providers
> - providerCache.put(cl, providers);
Re: svn commit: r924793 - in /geronimo/specs/trunk: geronimo-ejb_3.1_spec/
geronimo-el_2.2_spec/ geronimo-jacc_1.1_spec/ geronimo-jaspic_1.0_spec/ geronimo-jaxr_1.0_spec/
geronimo-jaxrpc_1.1_spec/ geronimo-jpa_2.0_spec/src/main/java/javax/persistence/
gero...
Posted by Rick McGuire <ri...@gmail.com>.
On 3/18/2010 10:49 PM, Donald Woods wrote:
> Rick, one comment on the geronimo-jpa_2.0_spec changes -
>
> I'm adding the following back in, as the JPA 2.0 Spec suggests using a
> provider cache per classloader and even requires implementing a
> public void clearCachedProviders()
> method to handle flushing the cache.
>
> Otherwise, initial quick scan of changes (w/o testing) looks good.
>
> I'll run the stand-alone JPA 2.0 TCK next week, after we get our OpenJPA
> 2.0.0 branch created.
>
Ah, thanks. That was an inadvertent deletion. I had intended on moving
that exactly to the place you put it back in.
Rick
>
> -Donald
>
>
> On 3/18/10 10:14 AM, rickmcguire@apache.org wrote:
>
>> -
>> - // cache the discovered providers
>> - providerCache.put(cl, providers);
>>
>