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

As NettyServerCnxn.sendResponse() allows all the exception to bubble up it can stop main ZK requests processing thread

Add voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Patch Available
    • Major
    • Resolution: Unresolved
    • 3.5.1
    • None
    • server

    Description

      As NettyServerCnxn.sendResponse() allows all the exception to bubble up it can stop main ZK requests processing thread and make Zookeeper server look like it is hanging, while it just can not process any request anymore.

      Idea is to catch all the exceptions in NettyServerCnxn.sendResponse() , convert them to IOException and allow it propagating up

      Attachments

        1. ZOOKEEPER-2549.patch
          14 kB
          Yuliya Feldman
        2. ZOOKEEPER-2549.patch
          14 kB
          Yuliya Feldman
        3. zookeeper-2549-1.patch
          23 kB
          Yuliya Feldman
        4. ZOOKEEPER-2549-2.patch
          24 kB
          Yuliya Feldman
        5. ZOOKEEPER-2549-3.patch
          25 kB
          Yuliya Feldman
        6. ZOOKEEPER-2549-3.patch
          24 kB
          Yuliya Feldman
        7. ZOOKEEPER-2549-4.patch
          22 kB
          Yuliya Feldman
        8. ZOOKEEPER-2549-5.patch
          23 kB
          Yuliya Feldman

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            yufeldman Yuliya Feldman
            yufeldman Yuliya Feldman

            Dates

              Created:
              Updated:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 10m
                10m

                Slack

                  Issue deployment