使用ScheduledExecutorService的

Question

我的Tomcat 7時的Java web應用程序的內存泄漏報道稱，有可能是內存泄漏在我的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. 

我在我的webapp一個長期運行的任務時，web應用程序啓動是被初始化。

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

} 

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

}

即使在關閉時服務器，它仍然報告有可能是內存泄漏我打電話scheduler.stop();。

這個應用程序部署在jelastic.com上，我發現它一旦啓動，它運行良好大約兩天，然後任務似乎沒有運行。日誌中也沒有例外或錯誤。

我在這裏做錯了什麼嗎？是否真的存在內存泄漏？

Answer 1

調用fScheduler.shutdownNow();是不夠的：

有沒有保證盡最大努力，試圖積極停止處理執行任務。

從JavaDoc。

相反，你必須明確地等待當前正在運行的任務：

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

Answer 2

我認爲你不應該調用來自監聽器，而是直接從servlet關機。

contextDestroyed()對於執行器服務來說已經太晚了。如javadoc 中所述，所有servlet和過濾器都將被銷燬之前任何ServletContextListeners都會收到關於上下文銷燬的通知。

而重寫的servlet destroy()應作爲根據的Javadoc 這種方法確定給servlet的一個機會，清理正在舉行的任何資源（例如，內存，文件句柄，線程 .. 。

@Override 
public void destroy() { 


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

    super.destroy(); 
}