Apache S4
  1. Apache S4
  2. S4-108

Share ZooKeeper client connections from S4 nodes

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.5.0
    • Fix Version/s: 0.6
    • Labels:
      None

      Description

      There are currently up to 5 or 6 connections that are initiated from various parts of S4, and this can lead to issues with the ZooKeeper servers. We can limit the number of connections by sharing them.

        Issue Links

          Activity

          Matthieu Morel created issue -
          Matthieu Morel made changes -
          Field Original Value New Value
          Link This issue is part of S4-114 [ S4-114 ]
          Hide
          Matthieu Morel added a comment -

          Addressed this issue in a patch in branch S4-114, commit [d6decd0]

          The idea is to inject ZkClient instances, built through a provider, and bind ZkClient with Singleton scope.

          Show
          Matthieu Morel added a comment - Addressed this issue in a patch in branch S4-114 , commit [d6decd0] The idea is to inject ZkClient instances, built through a provider, and bind ZkClient with Singleton scope.
          Hide
          Aimee Cheng added a comment -

          Looks good! And I also tried it, works well!

          +1

          Show
          Aimee Cheng added a comment - Looks good! And I also tried it, works well! +1
          Hide
          Matthieu Morel added a comment -

          developed in branch S4-114 and merged into dev in commit [6cd8742]

          Thanks for the feedback Aimee!

          Show
          Matthieu Morel added a comment - developed in branch S4-114 and merged into dev in commit [6cd8742] Thanks for the feedback Aimee!
          Matthieu Morel made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Matthieu Morel [ mmorel ]
          Resolution Fixed [ 1 ]
          Matthieu Morel made changes -
          Link This issue contains S4-107 [ S4-107 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          60d 5h 48m 1 Matthieu Morel 25/Jan/13 15:24

            People

            • Assignee:
              Matthieu Morel
              Reporter:
              Matthieu Morel
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development