Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-2272

Code clean up in ZooKeeperServer and KerberosName

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Won't Fix
    • Affects Version/s: 3.5.0
    • Fix Version/s: 3.5.10, 3.7.0
    • Component/s: server
    • Labels:
      None

      Description

      1. Following code in org.apache.zookeeper.server.ZooKeeperServer should be cleaned up. Some how it got missed in code review
        if ((System.getProperty("zookeeper.allowSaslFailedClients") != null)
          &&  (System.getProperty("zookeeper.allowSaslFailedClients").equals("true"))) {
        

        should be replaced with

        if(Boolean.getBoolean("zookeeper.allowSaslFailedClients"))
        
      2. Similar code clean up can be done in org.apache.zookeeper.server.auth.KerberosName

        Attachments

        1. ZOOKEEPER-2272-01.patch
          2 kB
          Mohammad Arshad

          Activity

            People

            • Assignee:
              arshad.mohammad Mohammad Arshad
              Reporter:
              arshad.mohammad Mohammad Arshad
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: