You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ju...@apache.org on 2014/01/29 05:33:06 UTC

svn commit: r1562352 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/EventGenerator.java

Author: jukka
Date: Wed Jan 29 04:33:06 2014
New Revision: 1562352

URL: http://svn.apache.org/r1562352
Log:
OAK-1332: Large number of changes to the same node can fill observation queue

Fix failing NodeReorderTest

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/EventGenerator.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/EventGenerator.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/EventGenerator.java?rev=1562352&r1=1562351&r2=1562352&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/EventGenerator.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/observation/EventGenerator.java Wed Jan 29 04:33:06 2014
@@ -157,9 +157,9 @@ public class EventGenerator {
                 // check for reordering of child nodes
                 if (OAK_CHILD_ORDER.equals(before.getName())) {
                     Set<String> beforeSet =
-                            newLinkedHashSet(after.getValue(NAMES));
-                    Set<String> afterSet =
                             newLinkedHashSet(before.getValue(NAMES));
+                    Set<String> afterSet =
+                            newLinkedHashSet(after.getValue(NAMES));
                     afterSet.retainAll(beforeSet);
                     beforeSet.retainAll(afterSet);
                     String[] beforeNames = beforeSet.toArray(STRING_ARRAY);