Java Webapp Speicherleck, wenn ScheduledExecutorService

Question

Mein Tomcat 7 verwendet, wird berichtet, dass es in meinem Webapp, die initialisiert wird, wenn die Webapp gestartet

SEVERE: The web application [/mywebapp] appears to have started a 
thread named [pool-1-thread-1] but has failed to stop it. This is 
very likely to create a memory leak. 

Ich habe eine lange laufende Aufgabe ein Speicherleck in meinem Webapp sein kann.

public class MyContextListener implements ServletContextListener{ 
Scheduler scheduler = null; 

public MyContextListener(){ 
    scheduler = new Scheduler(); 
} 

@Override 
public void contextDestroyed(ServletContextEvent arg0) { 
    scheduler.stop(); 
} 

@Override 
public void contextInitialized(ServletContextEvent arg0) { 
    scheduler.start(); 
} 

} 

.. und mein Scheduler.java

public class Scheduler { 
private final ScheduledExecutorService fScheduler; 

public Scheduler() { 
    fScheduler = Executors.newScheduledThreadPool(1); 
} 


public void start(){ 
    fScheduler.scheduleWithFixedDelay(new Runnable() { 

     @Override 
     public void run() { 
      //Perform some task 
     } 
    }, 1, 240, TimeUnit.MINUTES); 
} 

public void stop(){ 
    fScheduler.shutdownNow(); 
} 

}

Auch wenn ich scheduler.stop(); Aufruf, wenn Sie den Server herunterzufahren, seine immer noch da Berichterstattung ein Speicherverlust sein könnte.

Diese App ist auf jelastic.com im Einsatz und ich finde, dass es nach dem Start gut läuft für etwa zwei Tage und dann scheinen die Aufgaben nicht ausgeführt werden. Es gibt keine Ausnahmen oder Fehler in den Protokollen.

Mache ich hier etwas falsch? Gibt es wirklich ein mögliches Speicherleck?

Answer 1

fScheduler.shutdownNow(); Aufruf ist nicht genug:

Es gibt keine Garantien über Best-Effort versucht Verarbeitung aktiv zu stoppen Aufgaben ausführt.

Von JavaDoc.

Stattdessen müssen Sie explizit für die Aufgaben warten, die derzeit ausgeführt werden:

fScheduler.shutdownNow(); 
fScheduler.awaitTermination(10, TimeUnit.SECONDS); 

Answer 2

Ich glaube, Sie nicht das Herunterfahren von den Zuhörern anrufen sollten, sondern aus dem Servlet direkt.

contextDestroyed() des Zuhörers ist zu spät für den Executor-Service. Wie im javadoc angegeben, werden alle Servlets und Filter zerstörtvoralle ServletContextListeners werden von Kontext Zerstörung benachrichtigt.

während das Überschreiben der Servlet destroy() wie nach dem javadoc Diese Methode OK sein sollte, gibt dem Servlet die Möglichkeit, alle Ressourcen zu bereinigen, die (beispielsweise festgehalten werden, Speicher, Datei-Handles,Fäden ..

.

@Override 
public void destroy() { 


    fScheduler.shutdownNow(); 
    fScheduler.awaitTermination(10, TimeUnit.SECONDS); 

    super.destroy(); 
}