<div dir="ltr">Hi, <div><br></div><div>I&#39;m about to update the Netty subsystem to make some changes after the feedback we got from the WildFly team [1].</div><div>I&#39;m indenting to first address the concerns about the extra xml element, use a module attribute to point where the factory class is located, and then the threading tips. </div>
<div><br></div><div>They also recommended that we create a specific subsystem for SimplePush. </div><div>The advantage of this as I see it is that we would then have a schema for the configuration options for SimplePush (and also the SockJS configuration options). Without this we need to add a generic properties element to the Netty subsystem. </div>
<div><br></div><div>Another advantage would also be just having one single module instead of two like we currently have (one for the Netty subsystem and one for SimplePush).</div><div><br></div><div>One disadvantage might be if we later would like the Unified Push Server to be base on Netty it would also have to have a specific subsystem.</div>
<div><br></div><div>I wanted to people think is the right approach here for us?</div><div><br></div><div>[1] <a href="https://community.jboss.org/thread/229635;cid=1374049762089-667">https://community.jboss.org/thread/229635;cid=1374049762089-667</a></div>
</div>