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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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: [EMAIL PROTECTED] >> >> For additional commands, e-mail: [EMAIL PROTECTED] >> >> >> >> >> > >> > >> > >> > -- >> > Davanum Srinivas :: http://davanum.wordpress.com >> > >> >> >> > > -- Davanum Srinivas :: http://davanum.wordpress.com
