[jboss-as7-dev] JBAS-9268: OOM when uploading content through HTTP API

[Jason Greene]

Oh and thanks for the patch!

Sent from my iPhone

On Apr 15, 2011, at 7:40 AM, Jason Greene <jgreene at redhat.com> wrote:

> Thanks Heiko I'll take a look into this.
> 
> Standard ASL is fine. It's more keeping Deps manageable.
> 
> Sent from my iPhone
> 
> On Apr 15, 2011, at 6:51 AM, Heiko Braun <hbraun at redhat.com> wrote:
> 
>> 
>> 
>> 
>> I've run into a OOM issue when uploading content through the HTTP API again.
>> (https://issues.jboss.org/browse/JBAS-9268)
>> 
>> 
>> I did take a look at the current implementation and propose that we change it in the following way:
>> 
>> - read multipart contents from mime boundary
>> - replace the custom multipart stream implementation with a mature one (leans on apache commons fileupload)
>> 
>> I've got this changes  tested and verified in a custom branch:
>> https://github.com/heiko-braun/jboss-as/commits/out_of_memory
>> 
>> However before going ahead, I would like get some feedback from
>> 
>> a) the original author (Jonathan Pearlin. Welcome onboard btw)
>> b) Jason wrt the Apache License sources (See org.jboss.as.domain.http.server.multipart.asf.*)
>> 
>> 
>> Regards, Ike
>> 
>> 
>> 
>> 
>> _______________________________________________
>> jboss-as7-dev mailing list
>> jboss-as7-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
> 
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev