You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@storm.apache.org by bo...@apache.org on 2014/05/06 17:13:27 UTC

[2/4] git commit: Addressed review comments

Addressed review comments


Project: http://git-wip-us.apache.org/repos/asf/incubator-storm/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-storm/commit/3c89fb52
Tree: http://git-wip-us.apache.org/repos/asf/incubator-storm/tree/3c89fb52
Diff: http://git-wip-us.apache.org/repos/asf/incubator-storm/diff/3c89fb52

Branch: refs/heads/master
Commit: 3c89fb524f57152353334a60cfc6a2121286c702
Parents: 404bf25
Author: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Authored: Wed Apr 30 14:49:35 2014 +0000
Committer: Robert (Bobby) Evans <ev...@yahoo-inc.com>
Committed: Wed Apr 30 14:49:35 2014 +0000

----------------------------------------------------------------------
 storm-core/src/jvm/backtype/storm/task/TopologyContext.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-storm/blob/3c89fb52/storm-core/src/jvm/backtype/storm/task/TopologyContext.java
----------------------------------------------------------------------
diff --git a/storm-core/src/jvm/backtype/storm/task/TopologyContext.java b/storm-core/src/jvm/backtype/storm/task/TopologyContext.java
index 288fcd8..1dd6b7d 100644
--- a/storm-core/src/jvm/backtype/storm/task/TopologyContext.java
+++ b/storm-core/src/jvm/backtype/storm/task/TopologyContext.java
@@ -232,7 +232,7 @@ public class TopologyContext extends WorkerTopologyContext implements IMetricsCo
         }
 
         if (metric == null) {
-            throw new NullPointerException("Cannot register a null metric");
+            throw new IllegalArgumentException("Cannot register a null metric");
         }
         
         Map m1 = _registeredMetrics;