[ http://issues.apache.org/struts/browse/STR-2937?page=comments#action_38889 ] Peter Fassev commented on STR-2937: -----------------------------------
An iterator is a common language construct, which may be used every ware, for instance to iterate over huge HTML formatted messages in a blog. Please look at the current implementation of Tomcat. Once created, all Bodies of a JspContext are cached (and currently there may be up to JspContext 100 cached contexts), which may lead to large memory consumption. If you read the comments on sum bugzilla issues about this, you will find from one of the main tomcat creator, which simply proposes, not to use tags, which produces huge buffered output. This is simply horrible and unacceptable. That's why I have set the priority of this Issue to "Critical". > Iterate Tag uses EVAL_BODY_TAG and always buffers the body content - add > option to include the body > --------------------------------------------------------------------------------------------------- > > Key: STR-2937 > URL: http://issues.apache.org/struts/browse/STR-2937 > Project: Struts 1 > Issue Type: Improvement > Components: Taglibs > Affects Versions: 1.2.9 > Reporter: Peter Fassev > Priority: Critical > > The doStartTag() method of the logic:iterate Tag returns EVAL_BODY_TAG, which > is equal to EVAL_BODY_BUFFERED. This is in many cases (I think in 99%) very > inefficient, especially for large collections or large body contents. Please > add an optional parameter, for instance "bufferBody=true|false", where the > user can select, whether to include or to buffer the body of the tag. The > default value may be "true", to prevent inconsistencies with older versions > but I want to propose to make the default value "false" for the 1.3.x Version > and to document the change in the migration tutorial. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/struts/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
