Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 342785

Summary:

Disable code coverage for performance tests

Product:

[Eclipse Project] Platform

Reporter:

Dani Megert <daniel_megert>

Component:

Releng

Assignee:

Kim Moir <kim.moir>

Status:

RESOLVED FIXED

QA Contact:

Severity:

critical

Priority:

CC:

kim.moir, Olivier_Thomann

Version:

3.7

Target Milestone:

3.7 M7

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
patch	none
patch	none

Description Dani Megert

2011-04-14 01:17:01 EDT

According to Kim, we run code coverage for the performance tests. This is bad as it can affect performance and I'm not sure we also do it in the perf branch (if not, double bad).

We should disable code coverage for performance tests asap.

Comment 1 Kim Moir

2011-04-14 14:34:11 EDT

Created attachment 193282 [details]
patch

Comment 2 Kim Moir

2011-04-14 17:27:37 EDT

Fixed, tagged for I-build.

Comment 3 Kim Moir

2011-04-14 17:27:52 EDT

Also verified in test build.

Comment 4 Dani Megert

2011-04-15 03:01:18 EDT

Kim, was code coverage also enabled in the baseline?

Comment 5 Kim Moir

2011-04-15 08:17:41 EDT

No, this change wasn't backported to the baseline.

Comment 6 Kim Moir

2011-04-26 09:17:03 EDT

reopening

Comment 7 Kim Moir

2011-04-26 11:02:57 EDT

Created attachment 194066 [details]
patch

Comment 8 Kim Moir

2011-04-27 16:02:33 EDT

verified in I20110426-2000. The console log is clean of jacoco report generation calls, no report generation directories were created.