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

    https://github.com/apache/storm/pull/2208#discussion_r128345781
  
    --- Diff: storm-client/src/jvm/org/apache/storm/Config.java ---
    @@ -967,12 +967,6 @@
         public static final String STORM_ZOOKEEPER_SUPERACL = 
"storm.zookeeper.superACL";
     
         /**
    -     * The topology Zookeeper authentication scheme to use, e.g. "digest". 
Defaults to no authentication.
    -     */
    -    @isString
    -    public static final String 
STORM_ZOOKEEPER_TOPOLOGY_AUTH_SCHEME="storm.zookeeper.topology.auth.scheme";
    -
    -    /**
    --- End diff --
    
    -1 the problem with deleting these is that we don't get any config checks 
that it is a string.
    
    We could move it to a different file so that it is hidden from end users.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to