Hello Adrian, sorry, I saw cvs-commits first and wrote to you personally. I'll try to figure out. in isReadTimedOut( ctx ) isReadOnly( ctx ) is checked. Weird...
alex Monday, October 14, 2002, 2:07:48 AM, you wrote: AB> Hi, AB> Could one of the CMP experts check this fix? It is in 3.2 only. AB> Without the !isReadOnly() check, the preloaded readahead fields AB> were ignored. AB> Regards, AB> 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 AB> _________________________________________________________________ AB> Chat with friends online, try MSN Messenger: http://messenger.msn.com -- Best regards, Alex Loubyansky ------------------------------------------------------- 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
