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);
>>      
>