Skip to content

Commit c4e8568

Browse files
committed
support "," in timestamps in CMS mixed lines (concurrent mode failures) (fixes #135)
1 parent 0bdc64f commit c4e8568

File tree

2 files changed

+100
-78
lines changed

2 files changed

+100
-78
lines changed

src/main/java/com/tagtraum/perf/gcviewer/imp/DataReaderSun1_6_0.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ public class DataReaderSun1_6_0 extends AbstractDataReaderSun {
156156
// pattern looks always like "...[CMS<datestamp>..." or "...[CMS<timestamp>..."
157157
// the next line starts with " (concurrent mode failure)" which in earlier releases followed "CMS" immediately
158158
// the same can happen with "...ParNew<timestamp|datestamp>..."
159-
private static Pattern linesMixedPattern = Pattern.compile("(.*\\[(CMS|ParNew|DefNew|ASCMS|ASParNew))([0-9]+[-.].*)");
159+
private static Pattern linesMixedPattern = Pattern.compile("(.*\\[(CMS|ParNew|DefNew|ASCMS|ASParNew))([0-9]+[-.,].*)");
160160
// Matcher group of start of line
161161
private static final int LINES_MIXED_STARTOFLINE_GROUP = 1;
162162
// Matcher group of end of line
@@ -169,7 +169,7 @@ public class DataReaderSun1_6_0 extends AbstractDataReaderSun {
169169
// AdaptiveSizeStop: collection: 1
170170
// [PSYoungGen: 16420K->2657K(19136K)] 16420K->15919K(62848K), 0.0109211 secs] [Times: user=0.00 sys=0.00, real=0.01 secs]
171171
// -> to parse it, the first line must be split, and the following left out until the rest of the gc information follows
172-
private static final String ADAPTIVE_SIZE_POLICY_PATTERN_STRING = "(.*GC \\([a-zA-Z ]*\\)|.*GC)(?:[0-9.:]*.*)[ ]?AdaptiveSize.*";
172+
private static final String ADAPTIVE_SIZE_POLICY_PATTERN_STRING = "(.*GC \\([a-zA-Z ]*\\)|.*GC)(?:[0-9.,:]*.*)[ ]?AdaptiveSize.*";
173173
private static final Pattern adaptiveSizePolicyPattern = Pattern.compile(ADAPTIVE_SIZE_POLICY_PATTERN_STRING);
174174

175175
// -XX:+PrintAdaptiveSizePolicy combined with -XX:-UseAdaptiveSizePolicy (not using the policy, just printing)

0 commit comments

Comments
 (0)