svn commit: r1198370 - in /archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger: DefaultIndexMerger.java TemporaryGroupIndexCleaner.java

[olamy]

Author: olamy
Date: Sun Nov  6 13:50:40 2011
New Revision: 1198370

URL: http://svn.apache.org/viewvc?rev=1198370&view=rev
Log:
maven indexer cleanup only directory content so cleanup manually directory too

Modified:
    
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/DefaultIndexMerger.java
    
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/TemporaryGroupIndexCleaner.java

Modified: 
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/DefaultIndexMerger.java
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/DefaultIndexMerger.java?rev=1198370&r1=1198369&r2=1198370&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/DefaultIndexMerger.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/DefaultIndexMerger.java
 Sun Nov  6 13:50:40 2011
@@ -23,6 +23,7 @@ import org.apache.archiva.admin.model.ma
 import org.apache.archiva.common.plexusbridge.MavenIndexerUtils;
 import org.apache.archiva.common.plexusbridge.PlexusSisuBridge;
 import org.apache.archiva.common.plexusbridge.PlexusSisuBridgeException;
+import org.apache.commons.io.FileUtils;
 import org.apache.maven.index.NexusIndexer;
 import org.apache.maven.index.context.IndexingContext;
 import org.apache.maven.index.context.UnsupportedExistingLuceneIndexException;
@@ -130,6 +131,12 @@ public class DefaultIndexMerger
             {
                 indexer.removeIndexingContext( indexingContext, true );
             }
+            File directory = temporaryGroupIndex.getDirectory();
+            if ( directory != null && directory.exists() )
+            {
+                FileUtils.deleteDirectory( directory );
+            }
+            temporaryGroupIndexes.remove( temporaryGroupIndex );
         }
         catch ( IOException e )
         {

Modified: 
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/TemporaryGroupIndexCleaner.java
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/TemporaryGroupIndexCleaner.java?rev=1198370&r1=1198369&r2=1198370&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/TemporaryGroupIndexCleaner.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-base/archiva-indexer/src/main/java/org/apache/archiva/indexer/merger/TemporaryGroupIndexCleaner.java
 Sun Nov  6 13:50:40 2011
@@ -21,14 +21,12 @@ package org.apache.archiva.indexer.merge
 import org.apache.archiva.common.plexusbridge.PlexusSisuBridge;
 import org.apache.archiva.common.plexusbridge.PlexusSisuBridgeException;
 import org.apache.maven.index.NexusIndexer;
-import org.apache.maven.index.context.IndexingContext;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.scheduling.annotation.Scheduled;
 import org.springframework.stereotype.Service;
 
 import javax.inject.Inject;
-import java.io.IOException;
 import java.util.Date;
 
 /**
@@ -61,24 +59,8 @@ public class TemporaryGroupIndexCleaner
             // cleanup files older than 60 minutes 3600000
             if ( new Date().getTime() - temporaryGroupIndex.getCreationTime() 
> 3600000 )
             {
-                try
-                {
-                    IndexingContext context = 
indexer.getIndexingContexts().get( temporaryGroupIndex.getIndexId() );
-                    if ( context != null )
-                    {
-                        indexer.removeIndexingContext( context, true );
-                    }
-                    else
-                    {
-                        indexMerger.cleanTemporaryGroupIndex( 
temporaryGroupIndex );
-                    }
-                    indexMerger.getTemporaryGroupIndexes().remove( 
temporaryGroupIndex );
-                    log.debug( "remove directory {}", 
temporaryGroupIndex.getDirectory() );
-                }
-                catch ( IOException e )
-                {
-                    log.warn( "failed to remove directory:" + 
temporaryGroupIndex.getDirectory(), e );
-                }
+                indexMerger.cleanTemporaryGroupIndex( temporaryGroupIndex );
+
             }
         }
     }