[ 
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)

Reply via email to