[GitHub] storm pull request #2908: STORM-3276: Updated Flux to deal with storm local ...

[srdo]

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

    https://github.com/apache/storm/pull/2908#discussion_r237005496
  
    --- Diff: flux/flux-core/src/main/java/org/apache/storm/flux/Flux.java ---
    @@ -52,17 +52,22 @@
     public class Flux {
         private static final Logger LOG = LoggerFactory.getLogger(Flux.class);
     
    +    @Deprecated
    --- End diff --
    
    Sure, what I meant was if we're going to break the API anyway I think it 
makes sense to remove the options entirely, rather than deprecating them.


---