[GitHub] [james-project] vttranlina commented on a diff in pull request #1366: JMAP: Add another read level to get attachment metadata without reading the email body

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

vttranlina commented on code in PR #1366:
URL: https://github.com/apache/james-project/pull/1366#discussion_r1058711824


##########
mailbox/cassandra/src/main/java/org/apache/james/mailbox/cassandra/mail/CassandraMessageDAO.java:
##########
@@ -324,6 +324,7 @@ private Mono<byte[]> buildContentRetriever(FetchType fetchType, BlobId headerId,
         switch (fetchType) {
             case FULL:
                 return getFullContent(headerId, bodyId);
+            case ATTACHMENTS_METADATA:

Review Comment:
   If I am correct, we will have a separate method (eg: `getContent(headerId...)`) for `ATTACHMENTS_METADATA`. Correct?



-- 
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-unsubscribe@james.apache.org

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