Remove unused `getMetricsCollectorName` function Remove `getMetricsCollectorName` from the `MetricsCollector` as it is not used. Change-Id: I1be871afcb9470326ca2f5ee808be9e865083237
diff --git a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/FSMetricsCollector.java b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/FSMetricsCollector.java index bbbb7a4..cf71d5c 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/FSMetricsCollector.java +++ b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/FSMetricsCollector.java
@@ -154,11 +154,6 @@ } @Override - public String getMetricsCollectorName() { - return "filesystem-statistics"; - } - - @Override public ImmutableList<GitRepoMetric> availableMetrics() { return availableMetrics; }
diff --git a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitRefsMetricsCollector.java b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitRefsMetricsCollector.java index ab3e529..1882dc2 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitRefsMetricsCollector.java +++ b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitRefsMetricsCollector.java
@@ -85,11 +85,6 @@ } @Override - public String getMetricsCollectorName() { - return "repo-ref-statistics"; - } - - @Override public ImmutableList<GitRepoMetric> availableMetrics() { return availableMetrics; }
diff --git a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitStatsMetricsCollector.java b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitStatsMetricsCollector.java index 06b2c48..5f61c26 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitStatsMetricsCollector.java +++ b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/GitStatsMetricsCollector.java
@@ -118,9 +118,4 @@ public ImmutableList<GitRepoMetric> availableMetrics() { return availableMetrics; } - - @Override - public String getMetricsCollectorName() { - return "git-statistics"; - } }
diff --git a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/MetricsCollector.java b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/MetricsCollector.java index dcff8e8..7d13ad8 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/MetricsCollector.java +++ b/src/main/java/com/googlesource/gerrit/plugins/gitrepometrics/collectors/MetricsCollector.java
@@ -35,13 +35,6 @@ Consumer<HashMap<GitRepoMetric, Long>> populateMetrics); /** - * Returns the name of the metric collector. - * - * @return {@code String} with the metric collector name - */ - String getMetricsCollectorName(); - - /** * Returns the list of available metrics provided by the collector. * * @return {@code ImmutableList<GitRepoMetric>} with the {@link GitRepoMetric} provided by the
diff --git a/src/test/java/com/googlesource/gerrit/plugins/gitrepometrics/FakeMetricsCollector.java b/src/test/java/com/googlesource/gerrit/plugins/gitrepometrics/FakeMetricsCollector.java index 1a00761..705d7fe 100644 --- a/src/test/java/com/googlesource/gerrit/plugins/gitrepometrics/FakeMetricsCollector.java +++ b/src/test/java/com/googlesource/gerrit/plugins/gitrepometrics/FakeMetricsCollector.java
@@ -24,7 +24,6 @@ import org.eclipse.jgit.internal.storage.file.FileRepository; class FakeMetricsCollector implements MetricsCollector { - private final String prefix; private final GitRepoMetric fakeMetric1; private final GitRepoMetric fakeMetric2; @@ -37,17 +36,11 @@ } @Override - public String getMetricsCollectorName() { - return prefix + "-fake-metrics-collector"; - } - - @Override public ImmutableList<GitRepoMetric> availableMetrics() { return ImmutableList.of(fakeMetric1, fakeMetric2); } protected FakeMetricsCollector(String prefix) { - this.prefix = prefix; this.fakeMetric1 = new GitRepoMetric(prefix + "-fake-metric-1", "Fake metric 1", "Count"); this.fakeMetric2 = new GitRepoMetric(prefix + "-fake-metric-2", "Fake metric 2", "Count"); }