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 2020/12/23 13:08:20 UTC

[GitHub] [james-project] jeantil commented on pull request #281: JAMES-3477 Disable copy on write through system property

jeantil commented on pull request #281:
URL: https://github.com/apache/james-project/pull/281#issuecomment-750293565


   I don't know if we should document it: 
   This is really an internal implementation detail, should it really be configurable (adding configuration means the already complex code becomes even more complex)
   I only introduced the flag to let *us* more easily benchmark both implementations. 
   I feel the loosing implementation should simply be removed once we have the benchmarks results  as there is no point in leaving an inferior implementation in the code.
   I'll update the comment to reflect that instead.


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

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