Ard Schrijvers pushed to branch jackrabbit-2.10.1-patched at cms-community / 
hippo-jackrabbit-patches


Commits:
1abd7682 by Ard Schrijvers at 2016-07-21T20:45:05+02:00
REPO-1436 Jackrabbit in certain cases can index a Hippo document twice which is 
bad for performance

- - - - -


2 changed files:

- 
jackrabbit-api/src/main/java/org/apache/jackrabbit/api/stats/RepositoryStatistics.java
- 
jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java


Changes:

=====================================
jackrabbit-api/src/main/java/org/apache/jackrabbit/api/stats/RepositoryStatistics.java
=====================================
--- 
a/jackrabbit-api/src/main/java/org/apache/jackrabbit/api/stats/RepositoryStatistics.java
+++ 
b/jackrabbit-api/src/main/java/org/apache/jackrabbit/api/stats/RepositoryStatistics.java
@@ -34,7 +34,7 @@ public interface RepositoryStatistics {
         BUNDLE_WRITE_DURATION(true),
         BUNDLE_WRITE_AVERAGE(false),
         BUNDLE_CACHE_ACCESS_COUNTER(true),
-        BUNDLE_CACHE_SIZE_COUNTER(true),
+        BUNDLE_CACHE_SIZE_COUNTER(false),
         BUNDLE_CACHE_MISS_COUNTER(true),
         BUNDLE_CACHE_MISS_DURATION(true),
         BUNDLE_CACHE_MISS_AVERAGE(false),


=====================================
jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java
=====================================
--- 
a/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java
+++ 
b/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java
@@ -691,6 +691,10 @@ public class SearchIndex extends AbstractQueryHandler {
         // based on removed ids get affected aggregate root nodes
         retrieveAggregateRoot(removedIds, aggregateRoots);
 
+        // again remove any aggregateRoot nodes that are new
+        // and therefore already up-to-date
+        aggregateRoots.keySet().removeAll(addedIds);
+
         // update aggregates if there are any affected
         if (!aggregateRoots.isEmpty()) {
             Collection<Document> modified =



View it on GitLab: 
https://code.onehippo.org/cms-community/hippo-jackrabbit-patches/commit/1abd768202f045bd69ecfb9f2c238d7036ec51d3
_______________________________________________
Hippocms-svn mailing list
Hippocms-svn@lists.onehippo.org
https://lists.onehippo.org/mailman/listinfo/hippocms-svn

Reply via email to