axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Davanum Srinivas" <dava...@gmail.com>
Subject Re: Committing patch contributed for MIME message parser and serializer based on Apache mime4j [WSCOMMONS-387]
Date Wed, 19 Nov 2008 15:40:45 GMT
Oleg,

was not referring to you per se. I meant the promise of super fast
performance if we switch to Mime4j...No, i certainly did not imply
that you are on the hook for that. we would need to look at it and
work on it as a team (all of us).

thanks,
dims

On Wed, Nov 19, 2008 at 10:16 AM, Oleg Kalnichevski <olegk@apache.org> wrote:
> On Wed, 2008-11-19 at 08:39 -0500, Davanum Srinivas wrote:
>> Let's please keep it as a parallel optional implementation. just like
>> we have so many components with specific interfaces and multiple
>> implementations.
>
> Davanum
>
> The whole trouble is that Axiom currently does not allow for multiple
> implementations of MIME processing as it does not have an abstract API
> for that to start with.
>
>
>>  If/when we finally see the promised results, all we
>> need to do is switch the default to the other implementation.
>>
>
> I am not sure I understand exactly what kind of results you are
> referring to. The patch (1) provides an abstract API for MIME processing
> and (2) eliminates in memory buffering of SOAP envelops. The Mime4j
> based parser is currently ~15% slower than the existing implementation,
> but I _personally_ see this a reasonable trade-off for a smaller memory
> footprint and a better API. So, I do not think I made any promises I am
> still to deliver upon.
>
> Anyways, this patch is just not worth the trouble. Scrap it
>
> Oleg
>
>> thanks,
>> dims
>>
>> On Wed, Nov 19, 2008 at 8:15 AM, Davanum Srinivas <davanum@gmail.com> wrote:
>> > I am asking for the MIME4J implementation to be an option. Not the
>> > default. Default should be what is in right now.
>> >
>> > thanks,
>> > dims
>> >
>> > On Wed, Nov 19, 2008 at 5:41 AM, Paul Fremantle <pzfreo@gmail.com> wrote:
>> >>> Asankha et al
>> >>>
>> >>> The patch was meant as a replacement for the existing implementation.
It
>> >>> is pointless to have two MIME frameworks in Axiom. Just forget about
the
>> >>> patch.
>> >>>
>> >>> Oleg
>> >>
>> >> -1! I think we should evaluate this patch just like any other. I'm not
>> >> clear what Dims is asking I'm afraid, but I certainly don't want to
>> >> forget about the patch.
>> >>
>> >> Paul
>> >>
>> >> ---------------------------------------------------------------------
>> >> To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
>> >> For additional commands, e-mail: axis-dev-help@ws.apache.org
>> >>
>> >>
>> >
>> >
>> >
>> > --
>> > Davanum Srinivas :: http://davanum.wordpress.com
>> >
>>
>>
>>
>
>



-- 
Davanum Srinivas :: http://davanum.wordpress.com

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


Mime
View raw message