Arsnael commented on code in PR #2018:
URL: https://github.com/apache/james-project/pull/2018#discussion_r1494069538
##########
mailbox/store/src/main/java/org/apache/james/mailbox/store/search/ListeningMessageSearchIndex.java:
##########
@@ -132,7 +132,7 @@ protected Mono<Void> handleAdded(MailboxSession session,
Mailbox mailbox, Added
private Flux<MailboxMessage> retrieveMailboxMessages(MailboxSession
session, Mailbox mailbox, MessageRange range) {
return factory.getMessageMapper(session)
- .findInMailboxReactive(mailbox, range, FetchType.FULL, UNLIMITED);
+ .findInMailboxReactive(mailbox, range, FetchType.HEADERS,
UNLIMITED);
Review Comment:
Wouldn't that make the search on the mail body unavailable though? I don't
see why a client using James would find that acceptable?
--
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-unsubscr...@james.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org
For additional commands, e-mail: notifications-h...@james.apache.org
