Index: oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/compaction/LoggingGCMonitor.java =================================================================== --- oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/compaction/LoggingGCMonitor.java (revision 1765299) +++ oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/compaction/LoggingGCMonitor.java (working copy) @@ -61,8 +61,7 @@ } @Override - public void compacted( - long[] segmentCounts, long[] recordCounts, long[] compactionMapWeights) { + public void compacted() { } @Override Index: oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java =================================================================== --- oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java (revision 1765299) +++ oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java (working copy) @@ -116,8 +116,8 @@ } @Override - public void compacted(long[] segmentCounts, long[] recordCounts, long[] compactionMapWeights) { - gcMonitor.compacted(segmentCounts, recordCounts, compactionMapWeights); + public void compacted() { + gcMonitor.compacted(); } @Override @@ -130,12 +130,7 @@ checkNotNull(cacheManager); 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]); + gcMonitor.compacted(); cacheManager.evictOldGeneration(newGeneration); break; case FAILURE: Index: oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreGCMonitor.java =================================================================== --- oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreGCMonitor.java (revision 1765299) +++ oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreGCMonitor.java (working copy) @@ -75,7 +75,7 @@ } @Override - public void compacted(long[] segmentCounts, long[] recordCounts, long[] compactionMapWeights) { + public void compacted() { lastCompaction = clock.getTime(); } Index: oak-segment-tar/src/test/java/org/apache/jackrabbit/oak/segment/SegmentCompactionIT.java =================================================================== --- oak-segment-tar/src/test/java/org/apache/jackrabbit/oak/segment/SegmentCompactionIT.java (revision 1765299) +++ oak-segment-tar/src/test/java/org/apache/jackrabbit/oak/segment/SegmentCompactionIT.java (working copy) @@ -710,8 +710,8 @@ } @Override - public void compacted(long[] segmentCounts, long[] recordCounts, long[] compactionMapWeights) { - delegate.compacted(segmentCounts, recordCounts, compactionMapWeights); + public void compacted() { + delegate.compacted(); lastCompacted = System.currentTimeMillis(); }