[jira] Closed: (IMAP-10) Fetch command is not very efficient

["Robert Burrell Donkin (JIRA)" <se...@james.apache.org>]

     [ https://issues.apache.org/jira/browse/IMAP-10?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robert Burrell Donkin closed IMAP-10.
-------------------------------------

    Resolution: Fixed

Reasonable test coverage for FETCH so should be working OK. Will be more efficient once mailbox API is redesigned.

> Fetch command is not very efficient
> -----------------------------------
>
>                 Key: IMAP-10
>                 URL: https://issues.apache.org/jira/browse/IMAP-10
>             Project: JAMES Imap
>          Issue Type: Improvement
>            Reporter: Zsombor Gegesy
>            Assignee: Robert Burrell Donkin
>         Attachments: opt.patch
>
>
> Currently the implementation of the FETCH command is not very efficient. For example to retrieve one mail the following conversions will happens:
>  1, the backend creates a MimeMessage from a file/database blob/thin air/etc
>  2, the frontend converts into a byte array, with correct CRLF line endings. (I don't get, why it's needed, every incoming MimeMessage is already in this format, isn't it ?)
>  3, the frontend appends it into a StringBuffer in the FetchCommand class
>  4, after some String manipulation it gets sent over an InternetPrintWriter which checks also that all line endings in correct style.
>  I'm not sure that the whole thing is absolutly necessary. There is two other problem with the current code : 
>  1, it doesn't calculate the correct MessageResult flags, so the backend cant optimize out the unnecessary things.
>  2,  It doesn't parse the BODY[]<0.1024> style commands. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org