[ 
https://issues.apache.org/jira/browse/TRINIDAD-2046?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13001043#comment-13001043
 ] 

Xiaoming Shi commented on TRINIDAD-2046:
----------------------------------------

Yes :-)

For resources that are smaller than the MAX_BUFFER size, we can just allocate a 
buffer with the size of the resource. 

Here is the final patch of the parent bug: 
https://issues.apache.org/bugzilla/attachment.cgi?id=13689

> Copying data from inputStream to OuputStream needs appropriate buffer size
> --------------------------------------------------------------------------
>
>                 Key: TRINIDAD-2046
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-2046
>             Project: MyFaces Trinidad
>          Issue Type: Improvement
>          Components: Components
>    Affects Versions: 1.2.14-core , 2.0.0-beta-2
>            Reporter: Xiaoming Shi
>              Labels: performance
>
> In the files 
> ./trinidad-1.2.14/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/image/cache/FileSystemImageCache.java
>    line:955    
> ./trinidad-1.2.14/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/image/painter/ImageUtils.java
>                 line:303           
> The buffer size is fixed as 1024 bytes. With the size of the data varies, the 
> performance can be damaged a lot.
> We need an appropriate buffer size which depends on the size of the data to 
> be copied.
> This is the same as the Appache Bug 
> (https://issues.apache.org/bugzilla/show_bug.cgi?id=32546)

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to