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:41:41 GMT
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>


Mime
View raw message