chibenwa commented on code in PR #997:
URL: https://github.com/apache/james-project/pull/997#discussion_r870159563
##########
mailbox/store/src/main/java/org/apache/james/mailbox/store/mail/MessageMapper.java:
##########
@@ -207,7 +207,7 @@ default Mono<List<UpdatedFlags>>
resetRecentReactive(Mailbox mailbox) {
MessageMetaData copy(Mailbox mailbox, MailboxMessage original) throws
MailboxException;
default List<MessageMetaData> copy(Mailbox mailbox, List<MailboxMessage>
original) throws MailboxException {
- return original.stream()
+ return original.parallelStream()
Review Comment:
parrallelStream is a dangerous abstraction, we should not use it. Doing one
range at the time is likely enough BTW.
If you really want to do stuff parrallely migrate this wall chunk of code to
reactor.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
