2011/4/13 Steve Ebersole <st...@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 <st...@hibernate.org> > http://hibernate.org > _______________________________________________ > hibernate-dev mailing list > hibernate-dev@lists.jboss.org > https://lists.jboss.org/mailman/listinfo/hibernate-dev > _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev