Github user agresch commented on a diff in the pull request:
https://github.com/apache/storm/pull/2504#discussion_r160776702
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -1081,6 +1088,14 @@ public Nimbus(Map<String, Object> conf, INimbus
inimbus, IStormClusterState stor
BlobStore blobStore, TopoCache topoCache, ILeaderElector
leaderElector, IGroupMappingServiceProvider groupMapper)
throws Exception {
this.conf = conf;
+
+ this.metricsStore = null;
--- End diff --
Re-verified that a null MetricStore should be safe.
---
