Bug 152802 - [performance] BasicProfileAnalyzer is creating reports that is not used
Summary: [performance] BasicProfileAnalyzer is creating reports that is not used
Status: CLOSED FIXED
Alias: None
Product: WTP Webservices
Classification: WebTools
Component: wst.wsi (show other bugs)
Version: 1.5   Edit
Hardware: PC Windows XP
: P2 major (vote)
Target Milestone: 1.5.1 M151   Edit
Assignee: Jeffrey Liu CLA
QA Contact: David Lauzon CLA
URL:
Whiteboard:
Keywords: performance
Depends on:
Blocks:
 
Reported: 2006-08-03 18:12 EDT by Jeffrey Liu CLA
Modified: 2006-09-21 14:25 EDT (History)
0 users

See Also:

Attachments
Patch for wst.wsi (8.25 KB, patch)
2006-08-21 18:31 EDT, Jeffrey Liu CLA 		no flags Details | Diff
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 Jeffrey Liu CLA 2006-08-03 18:12:31 EDT 
Build: WTP 1.5.1 0803

In BasicProfileAnalyzer.validateConformance(), it creates a DefaultReport object which seems to contain WS-I conformance information. However, when the WS-I validator is run from the workbench, this information is not used.

In the case of a large workspace (1000 WSDL files), we are spending 26 sec (15%) creating this report. We should find ways to not create this report at the first place, this will be a huge performance improvement.
Comment 1 Jeffrey Liu CLA 2006-08-03 18:23:55 EDT 
To be more specific, here's where the time was spent:

org.eclipse.wst.wsi.internal.core.analyzer.BasicProfileAnalyzer.validateConformance() 
 129,155 72 % 
                org.eclipse.wst.wsi.internal.core.analyzer.BasicProfileAnalyzer.validateWSDL(ReportArtifact, WSDLValidator, String, WSDLDocument) 
 85,883 48 % 
                org.eclipse.wst.wsi.internal.core.report.impl.DefaultReporter.startReport() 
 25,977 15 % 
                 org.eclipse.wst.wsi.internal.core.report.impl.ReportImpl.getStartXMLString(String) 
 25,977 15 % 
                  org.eclipse.wst.wsi.internal.core.util.TestUtils.getXMLComment() 
 25,186 14 % 
                   org.eclipse.wst.wsi.internal.core.util.Utils.getInputStream(String) 
 25,116 14 % 
                    java.lang.ClassLoader.getResource(String) 
 9,613 5 % 
                    java.io.FileInputStream.<init>(String) 
 8,582 5 % 
                    java.lang.ClassLoader.getResourceAsStream(String) 
 6,719 4 %
Comment 2 Jeffrey Liu CLA 2006-08-16 12:37:29 EDT 
Changing priority to P2 because we want to fix this in 1.5.1
Comment 3 Jeffrey Liu CLA 2006-08-21 18:31:07 EDT 
Created attachment 48336 [details]
Patch for wst.wsi
Comment 4 Jeffrey Liu CLA 2006-08-23 11:36:17 EDT 
Patch reviewed by Dave.
Comment 5 Jeffrey Liu CLA 2006-08-23 11:38:26 EDT 
Fixed and released.
Comment 6 Jeffrey Liu CLA 2006-09-21 14:25:34 EDT 
verified and close.