Looking good so far. A few observations: The core.remoting.impl.mina package contains: CleanupNotifier - this is generic core not MINA only? ClientKeepAliveFactory - do we need this - I don't see the point? ServerKeepAliveFactory - do we need this? MessagingCodec - this should be generic too? Also we can probably rename NIOConnector and NIOSession to something more generic since they're not specific to NIO View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4157755#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...