Java webapp rò rỉ bộ nhớ khi sử dụng ScheduledExecutorService

Question

My Tomcat 7 được báo cáo rằng có thể có một rò rỉ bộ nhớ trong webapp của tôi

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. 

Tôi có một nhiệm vụ chạy dài trong webapp của tôi mà được khởi tạo khi webapp được bắt đầu.

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

} 

.. và Scheduler.java tôi

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

}

Mặc dù tôi gọi scheduler.stop(); khi tắt máy chủ, nó vẫn báo cáo có thể có một rò rỉ bộ nhớ.

Ứng dụng này được triển khai trên jelastic.com và tôi thấy rằng khi ứng dụng được bắt đầu, nó chạy tốt trong khoảng hai ngày và sau đó các tác vụ dường như không chạy. Cũng không có ngoại lệ hoặc lỗi trong nhật ký.

Tôi có làm gì sai ở đây không? Có thực sự là một rò rỉ bộ nhớ tiềm năng?

Answer 1

Calling fScheduler.shutdownNow(); là không đủ:

Có gì đảm bảo ngoài nhất nỗ lực cố gắng để ngăn chặn xử lý tích cực thực hiện nhiệm vụ.

Từ JavaDoc.

Thay vào đó bạn phải chờ đợi một cách rõ ràng cho các nhiệm vụ hiện đang chạy:

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

Answer 2

Tôi tin rằng bạn không nên gọi tắt từ Listener nhưng từ Servlet trực tiếp.

contextDestroyed() của người nghe đã quá muộn cho dịch vụ thi hành. Như đã nêu trong javadoc Tất cả các servlet và bộ lọc sẽ bị hủytrướcbất kỳ ServletContextListeners nào được thông báo về sự phá hủy ngữ cảnh.

trong khi trọng các servlet destroy() nên được OK như theo javadoc Phương pháp này cung cấp cho các servlet một cơ hội để làm sạch bất kỳ tài nguyên đang được tổ chức (ví dụ, bộ nhớ, xử lý tập tin,đề ..

@Override 
public void destroy() { 


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

    super.destroy(); 
}