Mailing-List: contact issues-help@cxf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cxf.apache.org
Date: Fri, 8 Jan 2016 22:10:40 +0000 (UTC)
From: "Alessio Soldano (JIRA)" <jira@apache.org>
To: issues@cxf.apache.org
Message-ID: <JIRA.12928623.1452256059000.62140.1452291040600@Atlassian.JIRA>
In-Reply-To: <JIRA.12928623.1452256059000@Atlassian.JIRA>
References: <JIRA.12928623.1452256059000@Atlassian.JIRA>
 <JIRA.12928623.1452256059595@arcas>
Subject: [jira] [Updated] (CXF-6739) Reduce memory pressure in
 org.apache.cxf.attachment.AttachmentDeserializer
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


     [ https://issues.apache.org/jira/browse/CXF-6739?page=3Dcom.atlassian.=
jira.plugin.system.issuetabpanels:all-tabpanel ]

Alessio Soldano updated CXF-6739:
---------------------------------
    Description: Back in 2013, the api (now core) went through a major refa=
ctoring to remove direct dependency to javax.mail: https://github.com/apach=
e/cxf/commit/6ae8f4afe219581938c0020612212faa3e770064 Custom code for attac=
hment deserialization was added, but unfortunately that seems to be writing=
 a lot of char[] to the memory, because of a lot of String instances being =
created for parsing the attachment inputstream into the headers TreeMap. Th=
e process could be optimized to avoid useless string / char[] copies. It al=
so seems that the special handling of lines starting with spaces there is o=
ver complicated and covering a condition that can never be true.  (was: Bac=
k in 2013, the api (now core) went through a major refactoring to remove di=
rect dependency to javax.mail: https://github.com/apache/cxf/commit/6ae8f4a=
fe219581938c0020612212faa3e770064 Custom code for attachment deserializatio=
n was added, but unfortunately that seems to be writing a lot of char[] to =
the memory, because of a lot of String instances being created for parsing =
the attachment inputstream into the headers TreeMap. The process could be o=
ptimized to avoid useless string / byte[] copies. It also seems that the sp=
ecial handling of lines starting with spaces there is over complicated and =
covering a condition that can never be true.)

> Reduce memory pressure in org.apache.cxf.attachment.AttachmentDeserialize=
r
> -------------------------------------------------------------------------=
-
>
>                 Key: CXF-6739
>                 URL: https://issues.apache.org/jira/browse/CXF-6739
>             Project: CXF
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 3.0.0-milestone1
>            Reporter: Alessio Soldano
>            Assignee: Alessio Soldano
>             Fix For: 3.1.5, 3.2.0
>
>
> Back in 2013, the api (now core) went through a major refactoring to remo=
ve direct dependency to javax.mail: https://github.com/apache/cxf/commit/6a=
e8f4afe219581938c0020612212faa3e770064 Custom code for attachment deseriali=
zation was added, but unfortunately that seems to be writing a lot of char[=
] to the memory, because of a lot of String instances being created for par=
sing the attachment inputstream into the headers TreeMap. The process could=
 be optimized to avoid useless string / char[] copies. It also seems that t=
he special handling of lines starting with spaces there is over complicated=
 and covering a condition that can never be true.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)