You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Nathan Beyer (JIRA)" <ji...@apache.org> on 2006/06/21 05:04:30 UTC

[jira] Commented: (HARMONY-630) Performance improvement for BufferedReader and LineNumberReader classes

    [ http://issues.apache.org/jira/browse/HARMONY-630?page=comments#action_12417024 ] 

Nathan Beyer commented on HARMONY-630:
--------------------------------------

Were the numbers posted backwards? The original values seems better than the patch values. Also, what's the unit of measure for these results? Seconds, milliseconds?

> Performance improvement for BufferedReader and LineNumberReader classes
> -----------------------------------------------------------------------
>
>          Key: HARMONY-630
>          URL: http://issues.apache.org/jira/browse/HARMONY-630
>      Project: Harmony
>         Type: Improvement

>   Components: Classlib
>     Reporter: Vladimir Strigun
>     Assignee: Nathan Beyer
>     Priority: Minor
>  Attachments: Harmony-630.patch
>
> Analysis of BufferedReader and LineNumberReader was made by our team. These classes are widely used and it's recommended, for instance, that any character IO should go through BufferedReader for better performance. Therefore it should perform well and should not itself be a bottleneck. 
> We have identified opportunity to increase performance of mentioned classes. Publicly available bench DaCapo[1] was used for performance measurement and we found that DaCapo:hsqldb benchmark performance can be improved within small modification of java.io.BufferedReader and java.io.LineNumberReader classes. Main idea of the patch (I will attach it soon) is to remove StringBuilder allocation from "malloc"-hot methods readLine. 
> Here is result of DaCapo execution with patched and original versions of classlib+J9:
> Build	Dacapo: fop	Dacapo: hsqldb	Dacapo: xalan	Dacapo: chart	
> original	712.020	27232.647	4201.449	22573.192	
> patched	715.152	26703.966	4161.018	22580.771	
> [1] http://www-ali.cs.umass.edu/DaCapo/gcbm.html

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira