From c2cb00d0bdd427bc151207056f7394826dfac1fb Mon Sep 17 00:00:00 2001
From: Nils Christian Ehmke <nie@informatik.uni-kiel.de>
Date: Thu, 29 Jan 2015 14:32:04 +0100
Subject: [PATCH] Minor bugfix
---
src/main/java/kieker/gui/common/model/DataModel.java | 7 -------
.../common/model/importer/ImportAnalysisConfiguration.java | 7 -------
.../model/importer/ImportAnalysisConfigurationTest.java | 1 -
3 files changed, 15 deletions(-)
diff --git a/src/main/java/kieker/gui/common/model/DataModel.java b/src/main/java/kieker/gui/common/model/DataModel.java
index 770c7356..a77941f9 100644
--- a/src/main/java/kieker/gui/common/model/DataModel.java
+++ b/src/main/java/kieker/gui/common/model/DataModel.java
@@ -26,7 +26,6 @@ import java.util.concurrent.TimeUnit;
import kieker.common.record.misc.KiekerMetadataRecord;
import kieker.gui.common.domain.AggregatedExecution;
import kieker.gui.common.domain.Execution;
-import kieker.gui.common.domain.Record;
import kieker.gui.common.model.importer.ImportAnalysisConfiguration;
import teetime.framework.Analysis;
@@ -37,7 +36,6 @@ import teetime.framework.Analysis;
*/
public final class DataModel extends Observable {
- private List<Record> records = Collections.emptyList();
private List<Execution> traces = Collections.emptyList();
private List<Execution> failureContainingTraces = Collections.emptyList();
private List<Execution> failedTraces = Collections.emptyList();
@@ -57,7 +55,6 @@ public final class DataModel extends Observable {
analysis.start();
// Store the results from the analysis
- this.records = analysisConfiguration.getRecordsList();
this.traces = analysisConfiguration.getTracesList();
this.failedTraces = analysisConfiguration.getFailedTracesList();
this.failureContainingTraces = analysisConfiguration.getFailureContainingTracesList();
@@ -110,10 +107,6 @@ public final class DataModel extends Observable {
return result;
}
- public List<Record> getRecordsCopy() {
- return new ArrayList<>(this.records);
- }
-
public List<Execution> getTracesCopy() {
return new ArrayList<>(this.traces);
}
diff --git a/src/main/java/kieker/gui/common/model/importer/ImportAnalysisConfiguration.java b/src/main/java/kieker/gui/common/model/importer/ImportAnalysisConfiguration.java
index 37f634e0..c8235d4b 100644
--- a/src/main/java/kieker/gui/common/model/importer/ImportAnalysisConfiguration.java
+++ b/src/main/java/kieker/gui/common/model/importer/ImportAnalysisConfiguration.java
@@ -25,7 +25,6 @@ import kieker.common.record.flow.IFlowRecord;
import kieker.common.record.misc.KiekerMetadataRecord;
import kieker.gui.common.domain.AggregatedExecution;
import kieker.gui.common.domain.Execution;
-import kieker.gui.common.domain.Record;
import kieker.gui.common.model.importer.stages.ReadingComposite;
import kieker.gui.common.model.importer.stages.TraceAggregationComposite;
import kieker.gui.common.model.importer.stages.TraceReconstructionComposite;
@@ -43,8 +42,6 @@ import teetime.stage.MultipleInstanceOfFilter;
*/
public final class ImportAnalysisConfiguration extends AnalysisConfiguration {
- private final List<Record> records = new Vector<>(1000);
-
private final List<Execution> traces = new Vector<>(1000);
private final List<Execution> failedTraces = new Vector<>(1000);
private final List<Execution> failureContainingTraces = new Vector<>(1000);
@@ -75,10 +72,6 @@ public final class ImportAnalysisConfiguration extends AnalysisConfiguration {
super.addThreadableStage(reader);
}
- public List<Record> getRecordsList() {
- return this.records;
- }
-
public List<Execution> getTracesList() {
return this.traces;
}
diff --git a/src/test/java/kieker/gui/common/model/importer/ImportAnalysisConfigurationTest.java b/src/test/java/kieker/gui/common/model/importer/ImportAnalysisConfigurationTest.java
index 830d6746..b99884f7 100644
--- a/src/test/java/kieker/gui/common/model/importer/ImportAnalysisConfigurationTest.java
+++ b/src/test/java/kieker/gui/common/model/importer/ImportAnalysisConfigurationTest.java
@@ -22,7 +22,6 @@ public class ImportAnalysisConfigurationTest {
analysis.start();
assertThat(configuration.getMetadataRecords(), hasSize(1));
- assertThat(configuration.getRecordsList(), is(not(empty())));
assertThat(configuration.getTracesList(), is(not(empty())));
assertThat(configuration.getAggregatedTraces(), is(not(empty())));
assertThat(configuration.getFailedTracesList(), is(not(empty())));
--
GitLab