Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 345778 - Duplicate tld entries in taglib index
Summary: Duplicate tld entries in taglib index
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: jst.jsp (show other bugs)
Version: 3.2.4   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 3.2.4+   Edit
Assignee: Nick Sandonato CLA
QA Contact: Nick Sandonato CLA
URL:
Whiteboard: WI74903
Keywords:
Depends on:
Blocks: 346024
  Show dependency tree
 
Reported: 2011-05-13 15:21 EDT by Nick Sandonato CLA
Modified: 2011-05-16 22:27 EDT (History)
4 users (show)

See Also:
thatnitind: review+

Attachments
patch (3.20 KB, patch)
2011-05-13 15:22 EDT, Nick Sandonato CLA 		no flags Details | Diff
patch for helios (3.20 KB, patch)
2011-05-16 11:58 EDT, Nick Sandonato CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Sandonato CLA 2011-05-13 15:21:27 EDT 
After taking a look at David's taglibindex from Bug 321602 it became apparent that TLDs were being repeated per-JAR in the taglib index.

Over time, the list grows. When restoring the references for a project description, one copy of the URLRecord was being added to the JARRecord's list. Each project could then contribute the same TLDs to back to that JARRecord. Over time, with projects feeding back superfluous entires into each other's records performance begins to degrade. Depending on the number of TLDs and the number of projects referencing those JARs, it may be awhile before any performance problems are noticed.

Easiest way to fix the problem is to delete the files from .metadata/.plugins/org.eclipse.jst.jsp.core/taglibindex
Comment 1 Nick Sandonato CLA 2011-05-13 15:22:00 EDT 
Created attachment 195628 [details]
patch
Comment 2 Nick Sandonato CLA 2011-05-16 11:58:10 EDT 
Created attachment 195750 [details]
patch for helios
Comment 3 Carl Anderson CLA 2011-05-16 21:23:44 EDT 
Committed to R3_2_4_patches