ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-881

ZooKeeperServer.loadData loads database twice

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3.2, 3.4.0
    • Component/s: server
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      zkDb.loadDataBase() is called twice at the beginning of loadData(). It shouldn't have any negative affects, but is unnecessary. A patch should be trivial.

      1. ZOOKEEPER-881.patch
        0.6 kB
        Jared Cantwell

        Activity

        Hide
        Flavio Junqueira added a comment -

        Committed to the 3.3 branch (Committed revision 1023935.)

        Show
        Flavio Junqueira added a comment - Committed to the 3.3 branch (Committed revision 1023935.)
        Hide
        Patrick Hunt added a comment -

        Reopening - this was only committed to trunk, slated for 3.3.2 and trunk.

        Show
        Patrick Hunt added a comment - Reopening - this was only committed to trunk, slated for 3.3.2 and trunk.
        Hide
        Flavio Junqueira added a comment -

        Ben forgot to close this issue.

        Show
        Flavio Junqueira added a comment - Ben forgot to close this issue.
        Hide
        Hudson added a comment -

        Integrated in ZooKeeper-trunk #967 (See https://hudson.apache.org/hudson/job/ZooKeeper-trunk/967/)
        ZOOKEEPER-881. ZooKeeperServer.loadData loads database twice (jared cantwell via breed)

        Show
        Hudson added a comment - Integrated in ZooKeeper-trunk #967 (See https://hudson.apache.org/hudson/job/ZooKeeper-trunk/967/ ) ZOOKEEPER-881 . ZooKeeperServer.loadData loads database twice (jared cantwell via breed)
        Hide
        Benjamin Reed added a comment -

        Committed revision 1022824.

        Show
        Benjamin Reed added a comment - Committed revision 1022824.
        Hide
        Benjamin Reed added a comment -

        +1 nice catch!

        Show
        Benjamin Reed added a comment - +1 nice catch!

          People

          • Assignee:
            Jared Cantwell
            Reporter:
            Jared Cantwell
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development