Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-835 Refactoring Zookeeper Client Code
  3. ZOOKEEPER-836

hostlist as string

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.3.1
    • None
    • java client
    • None
    • Reviewed
    • hostliste, comma seperated

    Description

      The hostlist is parsed in the ctor of ClientCnxn. This violates the rule of
      not doing (too much) work in a ctor. Instead the ClientCnxn should receive an
      object of class "HostSet". HostSet could then be instantiated e.g. with a
      comma separated string.

      Attachments

        1. ZOOKEEPER-836_2.patch
          28 kB
          Benjamin Reed
        2. ZOOKEEPER-836.patch
          29 kB
          Thomas Koch
        3. ZOOKEEPER-836.patch
          28 kB
          Thomas Koch
        4. ZOOKEEPER-836.patch
          31 kB
          Thomas Koch
        5. ZOOKEEPER-836.patch
          14 kB
          Thomas Koch

        Issue Links

          is blocked by

          Sub-task - The sub-task of the issue ZOOKEEPER-762 Allow dynamic addition/removal of server nodes in the client API

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          is related to

          New Feature - A new feature of the product, which has yet to be developed. ZOOKEEPER-338 zk hosts should be resolved periodically for loadbalancing amongst zk servers.

          • Major - Major loss of function.
          • Open

          New Feature - A new feature of the product, which has yet to be developed. ZOOKEEPER-781 provide a generalized "connection strategy" for ZooKeeper clients

          • Major - Major loss of function.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. ZOOKEEPER-146 Allow clients to retrieve the list of hosts/servers from a URL

          • Major - Major loss of function.
          • Open

          Activity

            People

              thkoch Thomas Koch
              ymc-pada Patrick Datko
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: