Removing unnecessary null check on STDOUT stream
Project: http://git-wip-us.apache.org/repos/asf/storm/repo Commit: http://git-wip-us.apache.org/repos/asf/storm/commit/137efb10 Tree: http://git-wip-us.apache.org/repos/asf/storm/tree/137efb10 Diff: http://git-wip-us.apache.org/repos/asf/storm/diff/137efb10 Branch: refs/heads/1.x-branch Commit: 137efb108c2084d1cf371c899d6cd454b9166882 Parents: 257f1d3 Author: Kishor Patil <[email protected]> Authored: Thu Feb 4 15:29:17 2016 -0600 Committer: Kishor Patil <[email protected]> Committed: Fri Feb 5 19:30:06 2016 +0000 ---------------------------------------------------------------------- .../daemon/metrics/reporters/ConsolePreparableReporter.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/storm/blob/137efb10/storm-core/src/jvm/org/apache/storm/daemon/metrics/reporters/ConsolePreparableReporter.java ---------------------------------------------------------------------- diff --git a/storm-core/src/jvm/org/apache/storm/daemon/metrics/reporters/ConsolePreparableReporter.java b/storm-core/src/jvm/org/apache/storm/daemon/metrics/reporters/ConsolePreparableReporter.java index 3ef4237..1eacb63 100644 --- a/storm-core/src/jvm/org/apache/storm/daemon/metrics/reporters/ConsolePreparableReporter.java +++ b/storm-core/src/jvm/org/apache/storm/daemon/metrics/reporters/ConsolePreparableReporter.java @@ -23,7 +23,6 @@ import org.apache.storm.daemon.metrics.MetricsUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import java.io.PrintStream; import java.util.Locale; import java.util.Map; import java.util.concurrent.TimeUnit; @@ -37,11 +36,7 @@ public class ConsolePreparableReporter implements PreparableReporter<ConsoleRepo LOG.debug("Preparing..."); ConsoleReporter.Builder builder = ConsoleReporter.forRegistry(metricsRegistry); - PrintStream stream = System.out; - if (stream != null) { - builder.outputTo(stream); - } - + builder.outputTo(System.out); Locale locale = MetricsUtils.getMetricsReporterLocale(stormConf); if (locale != null) { builder.formattedFor(locale);
