netty 使用注意事项

最近在使用netty的时候突然碰到这样的一个警告:

 

Java代码  

1. 2010-8-11 12:20:28 org.jboss.netty.util.internal.SharedResourceMisuseDetector  
2. 警告: You are creating too many MemoryAwareThreadPoolExecutor instances.  MemoryAwareThreadPoolExecutor is a shared resource that must be reused across the application, so that only a few instances are created.  
3. 2010-8-11 12:20:28 org.jboss.netty.util.internal.SharedResourceMisuseDetector  
4. 警告: You are creating too many HashedWheelTimer instances.  HashedWheelTimer is a shared resource that must be reused across the application, so that only a few instances are created.  

说的是我在使用

MemoryAwareThreadPoolExecutor和HashedWheelTimer

的时候创造了太多的实例.后来一看源码才发现问题所在!这两个貌似都是线程池的对象,在各自的构造方法里面,每实例一个对象就会使各自SharedResourceMisuseDetector(滥用共享资源探测器)加一.当超过256的时候就报警了!

Java代码  

1. private static final SharedResourceMisuseDetector misuseDetector =  
2.         new SharedResourceMisuseDetector(MemoryAwareThreadPoolExecutor.class);  
3. .....  
4. // Misuse check  
5.         misuseDetector.increase();  

 后来再查看HashedWheelTimer的源代码中还发现了这样的提示:

Java代码  

1. <h3>Do not create many instances.</h3>  
2.  *  
3.  * {@link HashedWheelTimer} creates a new thread whenever it is instantiated and  
4.  * started.  Therefore, you should make sure to create only one instance and  
5.  * share it across your application.  One of the common mistakes, that makes  
6.  * your application unresponsive, is to create a new instance in  
7.  * {@link ChannelPipelineFactory}, which results in the creation of a new thread  
8.  * for every connection.  

大致就说不要创建太多的实例

之前我是这样写的

Java代码  

1. public class ServerPipelineFactory implements ChannelPipelineFactory {  
2. ...  
3. @Override  
4.     public ChannelPipeline getPipeline() throws Exception {  
5. ChannelPipeline pipeline = pipeline();  
6. pipeline.addLast("executor", new ExecutionHandler(new OrderedMemoryAwareThreadPoolExecutor(16, 1048576, 1048576)));  
7. pipeline.addLast("timeout", new ReadTimeoutHandler(new HashedWheelTimer(), 10));  

 这样以来每个channel获取PipelineFactory的时候都会重新实例MemoryAwareThreadPoolExecutor和HashedWheelTimer,

当连接一多的时候就报警了!

 

根据这个提示我修改了ServerPipelineFactory,把他们做出单例的引用

 

Java代码  

1. public class ServerPipelineFactory implements ChannelPipelineFactory {  
2. ...  
3. static OrderedMemoryAwareThreadPoolExecutor e = new OrderedMemoryAwareThreadPoolExecutor(16, 0, 0);  
4. static HashedWheelTimer hashedWheelTimer = new HashedWheelTimer();  
5. static ExecutionHandler executionHandler = new ExecutionHandler(e);  
6. @Override  
7.     public ChannelPipeline getPipeline() throws Exception {  
8. ChannelPipeline pipeline = pipeline();  
9. pipeline.addLast("executor", executionHandler );  
10. pipeline.addLast("timeout", new ReadTimeoutHandler(hashedWheelTimer, 10));  

 

 

这样就不会再有SharedResourceMisuseDetector的警告了!