I really like the way you designed this: the PreloadFinderCommand acting 
as a decorator really makes things clearer. This is getting faster 
everytime one of us touches it.

Something I noticed, Bill - your changes only worked for defined 
finders. I've extended them so that they work for findAll and the 
regular autogenerated findByField stuff as well. To support this I 
changed the 'defined' member of JDBCPreloadFinderCommand to be
a JDBCFinderCommand (superclass of all finders) and made his 
setParameters method simply delegate to the embedded finder command. I 
also changed the name of the embedded finder to finderDelegate: I think 
that's more descriptive now.

I'm going to test that a bit more, then check in.

-danch

[EMAIL PROTECTED] wrote:

>   User: patriot1burke
>   Date: 01/06/21 14:57:22
> 


_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to