[VELOCITY-662] EventHandler.referenceInsert performance bottleneck

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: Engine
• Assignee: Unassigned
• Reporter: Byron Foster
• Created: 03/Jan/09 20:23
• Updated: 04/Jan/09 04:43
• Resolved: 04/Jan/09 04:43

Description

The times resulting from SimpleNode.literal, StrBuilder.append (the top two) and EventHandlerUtil.referenceInsert are due to the following line in ASTReference.render:

value = EventHandlerUtil.referenceInsert(rsvc, context, literal(), value);

Which accounts for nearly 25% of the time, Ouch! Not to mention the many number of StrBuilder objects created. (StrBuilder is called from within literal()) We are always taking the hit for this call even if there is no event handler installed. I havn't looked into it, but will probably just put a conditional around it.