Sanne for 3.6 cool, but be aware that we are redoing all of this ATM in
master
On 04/12/2011 06:15 PM, Sanne Grinovero wrote:
2011/4/13 Steve Ebersole<steve(a)hibernate.org>:
> On 04/12/2011 06:01 PM, Max Rydahl Andersen wrote:
>> This was my original implementation but it was changed to this explicit API with
the reasoning
>> that doing this caching should be an explicit act of intent and not just a simple
flag working globally.
>
> But isn't the user explicitly saying "cache them all" the same on this
> point? To me it really comes down to whether there is a valid use case
> for caching some, but not all.
+1 for all or nothing.
If it's possible / makes sense, and as I see the implementation is
checking anyway for file timestamps, I don't see why any more fine
grained control could be needed.
Should it be on by default?
(BTW looking at implementation it seems the IOStreams are never closed
- I'm making a patch)
>> Note, if more than just xml based metadata could be cached that would be awesome
- but that might be
>> better solved via something like AS7 annotation index mechanism.
>
> funny you should say that... ;)
>
> --
> Steve Ebersole<steve(a)hibernate.org>
>
http://hibernate.org
> _______________________________________________
> hibernate-dev mailing list
> hibernate-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/hibernate-dev
>
--
Steve Ebersole <steve(a)hibernate.org>
http://hibernate.org