Hi, Could one of the CMP experts check this fix? It is in 3.2 only.
Without the !isReadOnly() check, the preloaded readahead fields were ignored. Regards, Adrian >From: Adrian Brock <[EMAIL PROTECTED]> >To: [EMAIL PROTECTED] >Subject: [jboss-cvs] jboss/src/main/org/jboss/ejb/plugins/cmp/jdbc >JDBCLoadEntityCommand.java >Date: Sun, 13 Oct 2002 15:53:53 -0700 > > User: ejort > Date: 02/10/13 15:53:53 > > Modified: src/main/org/jboss/ejb/plugins/cmp/jdbc Tag: Branch_3_2 > JDBCLoadEntityCommand.java > Log: > Fix readahead and load-groups > > Revision Changes Path > No revision > > > No revision > > > 1.19.2.4 +6 -2 >jboss/src/main/org/jboss/ejb/plugins/cmp/jdbc/JDBCLoadEntityCommand.java > > Index: JDBCLoadEntityCommand.java > =================================================================== > RCS file: >/cvsroot/jboss/jboss/src/main/org/jboss/ejb/plugins/cmp/jdbc/JDBCLoadEntityCommand.java,v > retrieving revision 1.19.2.3 > retrieving revision 1.19.2.4 > diff -u -r1.19.2.3 -r1.19.2.4 > --- JDBCLoadEntityCommand.java 5 Oct 2002 10:18:08 -0000 1.19.2.3 > +++ JDBCLoadEntityCommand.java 13 Oct 2002 22:53:53 -0000 1.19.2.4 > @@ -40,7 +40,7 @@ > * @author <a href="mailto:[EMAIL PROTECTED]";>Justin Forder</a> > * @author <a href="mailto:[EMAIL PROTECTED]";>Dirk Zimmermann</a> > * @author <a href="mailto:[EMAIL PROTECTED]";>danch (Dan >Christopherson)</a> > - * @version $Revision: 1.19.2.3 $ > + * @version $Revision: 1.19.2.4 $ > */ > public class JDBCLoadEntityCommand { > private final JDBCStoreManager manager; > @@ -286,7 +286,11 @@ > // - it is already loaded > // - it is a read-only _already_loaded_ field that isn't timed >out yet > if( field.isPrimaryKeyMember() > - || ( field.isLoaded( ctx ) && !field.isReadTimedOut( ctx ) >)) { > + || ( field.isLoaded( ctx ) > + && ( !field.isReadOnly() || !field.isReadTimedOut( ctx >) ) > + ) > + ) > + { > fields.remove(); > } > } > > > > > >------------------------------------------------------- >This sf.net email is sponsored by:ThinkGeek >Welcome to geek heaven. >http://thinkgeek.com/sf >_______________________________________________ >jboss-cvs-commits mailing list >[EMAIL PROTECTED] >https://lists.sourceforge.net/lists/listinfo/jboss-cvs-commits _________________________________________________________________ Chat with friends online, try MSN Messenger: http://messenger.msn.com ------------------------------------------------------- This sf.net email is sponsored by:ThinkGeek Welcome to geek heaven. http://thinkgeek.com/sf _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development
