Java webapp wyciek pamięci podczas korzystania ScheduledExecutorService

Question

My Tomcat 7 donosi, że może być przeciek pamięci w moim webapp

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. 

Mam długie zadanie działa w moim webapp, który pobiera zainicjowany gdy webapp jest uruchomiona.

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(); 
} 

} 

.. i moja 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(); 
} 

}

Chociaż nazywając scheduler.stop(); podczas zamykania serwera, jego wciąż zgłoszenie nie może być wyciek pamięci.

Ta aplikacja jest wdrożona na stronie jelastic.com i stwierdzam, że po uruchomieniu działa dobrze przez około dwa dni, a następnie zadania nie wydają się działać. Nie ma wyjątków ani błędów w dziennikach.

Czy robię coś złego tutaj? Czy naprawdę istnieje potencjalny wyciek pamięci?

Answer 1

Wywołanie fScheduler.shutdownNow(); nie wystarczy:

Nie ma żadnych gwarancji poza best-effort próbuje aktywnie zatrzymać przetwarzanie wykonywania zadań.

Od JavaDoc.

Zamiast tego należy jawnie czekać do zadań, które są aktualnie uruchomione:

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

Answer 2

Uważam, że nie powinno wywołać zamknięcie od słuchacza, ale z Servlet bezpośrednio.

contextDestroyed() słuchacza jest za późno na usługę executora. Jak podano w javadoc Wszystkie serwlety i filtry zostaną zniszczoneprzedwszelkie obiekty ServletContextListeners są powiadamiane o zniszczeniu kontekstu.

natomiast nadrzędne serwlet destroy() powinno być OK, ponieważ zgodnie z javadoc Metoda ta daje serwlet możliwość oczyścić wszelkie środki, które odbywają się (na przykład, pamięć, uchwytów plików,nici .. .

@Override 
public void destroy() { 


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

    super.destroy(); 
}