Summary: | [results] NPE while generating 3.3 performance results | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Frederic Fusier <frederic_fusier> | ||||
Component: | Releng | Assignee: | Frederic Fusier <frederic_fusier> | ||||
Status: | RESOLVED FIXED | QA Contact: | |||||
Severity: | normal | ||||||
Priority: | P3 | CC: | kim.moir | ||||
Version: | 3.3 | Keywords: | performance, test | ||||
Target Milestone: | 3.3.2 | ||||||
Hardware: | PC | ||||||
OS: | Windows XP | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Frederic Fusier
2007-10-23 09:48:02 EDT
The problem happens while cleaning tests with several measures in database per build. I missed the point that some values might be equals to 0 and in this case, the value is not stored in the specific 'values' array => one entry is null and we got this NPE while cleaning these values into one. I'm currently testing a patch to fix this issue... Created attachment 80952 [details]
Proposed patch
This patch fixes the problem by initializing values with -1 instead of 0...
Released for 3.4M3. I generated results without any troubles for M20071010-0800 and N20071015-0010 builds. I also tested files generated with this change: 1) N20071015-0010: they are *all* the same than those generated with version v20071023. 2) M20071010-0800: visual comparison with old version is OK Released new org.eclipse.test.performance jar to basebuilder for upcoming 3.4 builds. Would it be also taken into account for 3.3 builds or should I also release the fix in R3_3_maintenance branch? Yes, please release to the R3_3_maintenance branch branch. We have separate builders for each stream of builds. I'll update basebuilder with the new jar once it is available. Thanks! Reduce the severity as the fix for bug 201469 was not released in R3_3_maintenance branch => generation will not fail for 3.3 builds... Patch released in R3_3_maintenance stream basebuilder tagged with new performance plugins for next 3.3.1 stream M build (In reply to comment #9) > basebuilder tagged with new performance plugins for next 3.3.1 stream M build > Kim, do not forget to change the arguments as well: 1) the 'scenario.pattern' arguments need a '%' at the end to match all scenarios: -scenario.pattern org.eclipse.%.test%" 2) the order of config names is now used to determine the order of the fingerprint displayed in PHP pages: -config eclipseperflnx3_R3.3,eclipseperfwin2_R3.3,eclipseperflnx2_R3.3, eclipseperfwin1_R3.3,eclipseperflnx1_R3.3 3) the 'dataDir' argument must be added to use the incremental version and reduce the generation time to the minimum: -dataDir ${postingDirectory}/data/v33 WARNING: the directory *must* be located at a different place than the one used for the 3.4 builds results generation! Thanks thanks, I've released those fixes to the 3.3stream builder. |