[
https://issues.apache.org/jira/browse/STORM-235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13984559#comment-13984559
]
ASF GitHub Bot commented on STORM-235:
--------------------------------------
Github user ptgoetz commented on the pull request:
https://github.com/apache/incubator-storm/pull/96#issuecomment-41707553
+1
`IllegalArgumentException` seems more appropriate, IMHO, but the end result
is the same.
> Registering a null metric should blow up early
> ----------------------------------------------
>
> Key: STORM-235
> URL: https://issues.apache.org/jira/browse/STORM-235
> Project: Apache Storm (Incubating)
> Issue Type: Sub-task
> Reporter: Robert Joseph Evans
> Assignee: Robert Joseph Evans
>
> Registering a null metric should blow up when it is registered, not when it
> is queried.
--
This message was sent by Atlassian JIRA
(v6.2#6252)