commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Scott Sanders" <ssand...@nextance.com>
Subject RE: [Digester] Performance Improvements
Date Sat, 07 Sep 2002 00:54:56 GMT
Sorry, wrong page reference.  I meant this one:
http://jakarta.apache.org/site/source.html

Scott

> -----Original Message-----
> From: Scott Sanders 
> Sent: Friday, September 06, 2002 5:42 PM
> To: Jakarta Commons Developers List
> Subject: RE: [Digester] Performance Improvements
> 
> 
> Looking forward to the patch :)
> 
> 
> 
> http://jakarta.apache.org/site/bugs.html
> 
> Basically, a cvs diff -u as an attatchment sent to this 
> mailing list with a subject starting with [PATCH].
> 
> Scott
> 
> > -----Original Message-----
> > From: Gregory M. Messner [mailto:gmessner@titan.com]
> > Sent: Friday, September 06, 2002 5:06 PM
> > To: Jakarta Commons Developers List
> > Subject: [Digester] Performance Improvements
> > 
> > 
> > First off, I think the digester package is great, however 
> it is a bit
> > slow, poking around I noticed that each time a method was 
> invoked by 
> > CallMethodRule, SetNextRule, SetRootRule, and  SetTopRule the 
> > java.lang.reflect.Method was looked up each time. I made a 
> > small change 
> > in each of these classes that uses 
> > MethodUtils.getAccessibleMethod() or 
> > MethodUtils.getMatchingAccessibleMethod() (depending on the 
> > useExactMatch flag) to get the java.lang.reflect.Method 
> > instance, saves 
> > it and then directly calls invoke() on the Method instance.  On a 
> > SunBlade 100 this small change resulted in a 10X speed improvement.
> > 
> > I have created patch files for the changes but wasn't sure how to
> > deliver them, should they all be put in the body of a single 
> > email, each 
> > one sent in its own email, or should they be attached to an 
> > email?  As 
> > soon as someone lets me know the correct way to submit the 
> > patches I'll 
> > send them in.
> > 
> > Greg
> > 
> > 
> > --
> > To unsubscribe, e-mail:   
> > <mailto:commons-dev-> unsubscribe@jakarta.apache.org>
> > For
> > additional commands, 
> > e-mail: <mailto:commons-dev-help@jakarta.apache.org>
> > 
> > 
> 
> --
> To unsubscribe, e-mail:   
> <mailto:commons-dev-> unsubscribe@jakarta.apache.org>
> For 
> additional commands, 
> e-mail: <mailto:commons-dev-help@jakarta.apache.org>
> 
> 

--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message