[ 
https://issues.apache.org/jira/browse/STORM-2020?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15408026#comment-15408026
 ] 

ASF GitHub Bot commented on STORM-2020:
---------------------------------------

Github user HeartSaVioR commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1606#discussion_r73553509
  
    --- Diff: 
external/storm-hdfs/src/test/java/org/apache/storm/hdfs/spout/TestHdfsSpout.java
 ---
    @@ -704,12 +703,12 @@ public MockCollector() {
     
         @Override
         public void emitDirect(int arg0, String arg1, List<Object> arg2, 
Object arg3) {
    -      throw new NotImplementedException();
    +      throw new RuntimeException("NOT Implemented");
    --- End diff --
    
    I prefer using UnsupportedOperationException in this case but since  it's 
just a test code I'm OK.


> Stop using sun internal classes
> -------------------------------
>
>                 Key: STORM-2020
>                 URL: https://issues.apache.org/jira/browse/STORM-2020
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-hdfs, storm-redis
>    Affects Versions: 2.0.0
>            Reporter: Robert Joseph Evans
>            Assignee: Robert Joseph Evans
>
> sun.reflect.generics.reflectiveObjects.NotImplementedException, 
> sun.misc.BASE64Decoder, and sun.misc.BASE64Encoder are not public APIs we 
> should not be using them.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to