[GitHub] [james-project] jeantil commented on pull request #282: JAMES-3477 Drop MimeMessageCopyOnWriteProxy

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

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


   >MimeMessageInputStreamSource (holding the reference to the deferedFile) is disposed, each references to opened stream get cleaned up, and the file is *deleted*. We should be good...
   Thats what I meant : I may be reading  MimeMessageInputStreamSource's code incorrectly but I don't see a reason that dispose cannot be called by a thread `t1` while another thread `t2` is trying to read the contents of one of the inputstream returned by `getInputStream()`
   `t2` is going to get `java.io.IOException: inputstream is closed` because `t1` disposed the message and we are back to data loss territory. 
   


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