[ 
https://issues.apache.org/jira/browse/JAMES-3505?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17290658#comment-17290658
 ] 

Benoit Tellier commented on JAMES-3505:
---------------------------------------

This is a really good catch!

Looking at it it makes complete sense.

> However, I'm not sure if this will cause threading issues (maybe it's 
> supposed to be single threaded).

I guess there should not be any bad impact to what you propose. Delegating 
concurrency control to reactor is a good practice I would say.

Could you open a pull request regarding this?

> RemoteDelivery mailet single threaded for mail delivery
> -------------------------------------------------------
>
>                 Key: JAMES-3505
>                 URL: https://issues.apache.org/jira/browse/JAMES-3505
>             Project: James Server
>          Issue Type: Bug
>          Components: Remote Delivery
>    Affects Versions: master, 3.5.0
>            Reporter: Sjoerd de Feijter
>            Priority: Major
>
> RemoteDelivery mailet appears to be delivering mail using a single thread. 
> This affects throughput of email delivery significantly. 
> I'm not very familiar with Apache James but it looks like the cause of this 
> is the processMail method in DeliveryRunnable. The method is blocking and is 
> preventing multi-threaded behavior. Wrapping the contents of the method in a 
> Mono.create speeds up delivery significantly.
> However, I'm not sure if this will cause threading issues (maybe it's 
> supposed to be single threaded). So any input appreciated.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to