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 md...@apache.org on 2016/06/07 07:43:44 UTC

svn commit: r1747166 - /jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java

Author: mduerig
Date: Tue Jun  7 07:43:44 2016
New Revision: 1747166

URL: http://svn.apache.org/viewvc?rev=1747166&view=rev
Log:
OAK-4277: Finalise de-duplication caches
Fix session refresh on gc test for segment-tar

Modified:
    jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java

Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java?rev=1747166&r1=1747165&r2=1747166&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java Tue Jun  7 07:43:44 2016
@@ -116,6 +116,12 @@ public class FileStoreBuilder {
             public void compacted(@Nonnull Status status, final int newGeneration) {
                 switch (status) {
                     case SUCCESS:
+                        // FIXME OAK-4283: Align GCMonitor API with implementation
+                        // This call is still needed to ensure upstream consumers
+                        // of GCMonitor callback get properly notified. See
+                        // RepositoryImpl.RefreshOnGC and
+                        // LuceneIndexProviderService.registerGCMonitor().
+                        gcMonitor.compacted(new long[0], new long[0], new long[0]);
                         evictCaches(new Predicate<Integer>() {
                             @Override
                             public boolean apply(Integer generation) {