You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by GitBox <gi...@apache.org> on 2022/03/01 07:43:53 UTC

[GitHub] [james-project] ff-wl commented on pull request #899: JAMES-3715 Small cleanups following Netty 4 migration

ff-wl commented on pull request #899:
URL: https://github.com/apache/james-project/pull/899#issuecomment-1055118711


   > Cc @ff-wl I'm interested by your feedback...
   > 
   > Playing around with the code resulting from the Netty 4 migration ( #886 ) I realized:
   > 
   > * That the threading model was further simplified than I thought (which is good!) and that we could get rid of a few configuration options and of a few extra classes (yeah!)
   > * That some ssl related options were leftover
   > * And that the ioWorker option was no longer applied while we easily could.
   > 
   > @ff-wl feel free to cherry-pick the commits.
   
   looks good for me. I've cherry-picked all commits and will push them to the #886 branch.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org