[GitHub] [james-project] Arsnael edited a comment on pull request #932: JAMES-3732 Turn Netty leak detection to paranoid in protocols unit test

[GitBox <gi...@apache.org>]

Arsnael edited a comment on pull request #932:
URL: https://github.com/apache/james-project/pull/932#issuecomment-1077036942


   As I commented in the respective JIRA ticket... it's cool but... adding just paranoid level for leak detection in netty in tests... just for the pleasure of it.... if we don't find a way to fail the test when such a leak is detected I hardly see the point... Except for making our logs potentially bigger and our builds longer.
   
   Well except if someone accepts to check manually the full log of each build for leak detections? 


-- 
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