Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-1502

Remove need for heartbeats in HBase

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

Details

    • Task
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • None
    • 0.92.0
    • None
    • None
    • Reviewed
    • Hide
      Deprecates what were once considered fundamental classes, HServerInfo and HServerAddress. The former has been made irrelevant because its constituent info -- server name, server load -- is now passed otherwise. HServerAddress, a class that on deserialization, did a resolve (it created a new InetSocketAddress on deserialization) was just a bad idea from the get-go. Unfortunately, HSI and HSA were all over our codebase like a cancer. Hopefully the patient survives their purging.
      Show
      Deprecates what were once considered fundamental classes, HServerInfo and HServerAddress. The former has been made irrelevant because its constituent info -- server name, server load -- is now passed otherwise. HServerAddress, a class that on deserialization, did a resolve (it created a new InetSocketAddress on deserialization) was just a bad idea from the get-go. Unfortunately, HSI and HSA were all over our codebase like a cancer. Hopefully the patient survives their purging.

    Description

      HBase currently uses heartbeats between region servers and the master, piggybacking information on them when it can. This issue is to investigate if we can get rid of the need for those using ZooKeeper events.

      Attachments

        1. 1502.txt
          39 kB
          Michael Stack
        2. 1502-v2.txt
          126 kB
          Michael Stack
        3. 1502-4.txt
          332 kB
          Michael Stack
        4. 1502-v5.txt
          427 kB
          Michael Stack
        5. 1502-v6.txt
          427 kB
          Michael Stack
        6. 1502-v7.txt
          426 kB
          Michael Stack

        Issue Links

        Activity

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

          People

            stack Michael Stack
            nitay Nitay Joffe
            Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment