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

ReplicationZookeeper goes to ZK every time a znode is modified

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.90.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      While debugging other issues, I found that ReplicationAdmin.ReplicationStatusTracker is doing a ZK request every time a znode changes. Also reading the replication state shouldn't go to ZK if we already maintain a local variable that's updated with the tracker.

      1. HBASE-3351_3326.patch
        12 kB
        Jean-Daniel Cryans

        Activity

        Hide
        lars_francke Lars Francke added a comment -

        This issue was closed as part of a bulk closing operation on 2015-11-20. All issues that have been resolved and where all fixVersions have been released have been closed (following discussions on the mailing list).

        Show
        lars_francke Lars Francke added a comment - This issue was closed as part of a bulk closing operation on 2015-11-20. All issues that have been resolved and where all fixVersions have been released have been closed (following discussions on the mailing list).
        Hide
        hudson Hudson added a comment -

        Integrated in HBase-TRUNK #1697 (See https://hudson.apache.org/hudson/job/HBase-TRUNK/1697/)

        Show
        hudson Hudson added a comment - Integrated in HBase-TRUNK #1697 (See https://hudson.apache.org/hudson/job/HBase-TRUNK/1697/ )
        Hide
        jdcryans Jean-Daniel Cryans added a comment -

        Committed to branch and trunk, thanks for looking at it Stack.

        Show
        jdcryans Jean-Daniel Cryans added a comment - Committed to branch and trunk, thanks for looking at it Stack.
        Hide
        stack stack added a comment -

        +1 Looks reasonable to me J-D.

        Show
        stack stack added a comment - +1 Looks reasonable to me J-D.
        Hide
        jdcryans Jean-Daniel Cryans added a comment -

        Attaching a patch that covers both this issue and HBASE-3326 (Replication state's znode should be created else it defaults to false) because they are very intertwined.

        In this patch I fix HBASE-3326 simply by creating the znode and then I add the better tracking of the state znode which had an impact in more user-side classes since they had to provide an Abortable in order to be able to start the tracker in any given situation (before the client wasn't tracking the state znode). In the end, I think this is much better.

        Show
        jdcryans Jean-Daniel Cryans added a comment - Attaching a patch that covers both this issue and HBASE-3326 (Replication state's znode should be created else it defaults to false) because they are very intertwined. In this patch I fix HBASE-3326 simply by creating the znode and then I add the better tracking of the state znode which had an impact in more user-side classes since they had to provide an Abortable in order to be able to start the tracker in any given situation (before the client wasn't tracking the state znode). In the end, I think this is much better.

          People

          • Assignee:
            jdcryans Jean-Daniel Cryans
            Reporter:
            jdcryans Jean-Daniel Cryans
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development