[ http://jira.jboss.com/jira/browse/JBRULES-1173?page=comments#action_12375526 ] Mark Proctor commented on JBRULES-1173: --------------------------------------- actually I believe this is already fix, but slightly differently. Have you tried this with 4.0.1 to see if you can still get the error to occur? -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBRULES-1173?page=comments#action_12375538 ] Mark Proctor commented on JBRULES-1173: --------------------------------------- I've looked more at this, it doesn't seem a zero index problem, which makes me worry if factsize is being calculated correctly. I would really need code to reproduce this problem, so I can see exactly what it is. index < result.length is fixing the symptom not the cause, so we need to fix the cause. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBRULES-1173?page=all ] Nageshwar Kunichetty updated JBRULES-1173: ------------------------------------------ Workaround Description: So I had the following change in file "TupleIndexHashTable.java" and it was working : public Entry[] toArray() { Entry[] result = new Entry[this.factSize]; int index = 0; System.out.println("this.table.length=" + this.table.length); for ( int i = 0; i < this.table.length; i++ ) { FieldIndexEntry fieldIndexEntry = (FieldIndexEntry)this.table[i]; while ( fieldIndexEntry != null ) { Entry entry = fieldIndexEntry.getFirst(); //while ( entry != null){ while ( entry != null && index<result.length) { result[index++] = entry; entry = entry.getNext(); } fieldIndexEntry = ( FieldIndexEntry ) fieldIndexEntry.getNext(); } } return result; } That is, in innner while loop I added extra codition ("index<result.length") of "while ( entry != null && index<result.length) {". Fix worked for me. I am not sure if this is already fixed. Some I was wondering if it is fixed could I know the fixed release version/patch? Thanks. was: So I had the following change in file "TupleIndexHashTable.java" and it was working : public Entry[] toArray() { // Entry[] result = new Entry[this.factSize]; Entry[] result = new Entry[this.size]; int index = 0; System.out.println("this.table.length=" + this.table.length); for ( int i = 0; i < this.table.length; i++ ) { FieldIndexEntry fieldIndexEntry = (FieldIndexEntry)this.table[i]; while ( fieldIndexEntry != null ) { Entry entry = fieldIndexEntry.getFirst(); //while ( entry != null){ while ( entry != null && index<result.length) { result[index++] = entry; entry = entry.getNext(); } fieldIndexEntry = ( FieldIndexEntry ) fieldIndexEntry.getNext(); } } return result; } That is, in innner while loop I added extra codition ("index<result.length") of "while ( entry != null && index<result.length) {". Fix worked for me. I am not sure if this is already fixed. Some I was wondering if it is fixed could I know the fixed release version/patch? Thanks. It was wrong piece of code pasted I have corrected it. Only fix I did was following : That is, in innner while loop I added extra codition ("index<result.length") of "while ( entry != null && index<result.length) {". Fix worked for me. I am not sure if this is already fixed. Some I was wondering if it is fixed could I know the fixed release version/patch? I will send the test case tomorrow. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBRULES-1173?page=comments#action_12375809 ] Mark Proctor commented on JBRULES-1173: --------------------------------------- My only problem with this fix is that in theory this.factSize is the total number of available facts, and thus entry.getNext() should never be able to find any more than that. If we are also doing this index<result.length check this will be fixing the symptom not the cause. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBRULES-1173?page=comments#action_12376517 ] Mark Proctor commented on JBRULES-1173: --------------------------------------- nageshwar, we really want to put out a 4.0.2 this week, we would like to have this solved, but cannot do this without your example code. Any chance we could have this asap? Thanks Mark -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBRULES-1173?page=all ] Mark Proctor closed JBRULES-1173. --------------------------------- Resolution: Cannot Reproduce Bug user cannot re-produce with bug 4.0.3. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira