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

        Jared Cantwell created issue -
        Jared Cantwell made changes -
        Field Original Value New Value
        Summary ZooKeeperServer.loadData loads database twise ZooKeeperServer.loadData loads database twice
        Jared Cantwell made changes -
        Attachment ZOOKEEPER-881.patch [ 12457197 ]
        Jared Cantwell made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Assignee Jared Cantwell [ jaredc ]
        Patrick Hunt made changes -
        Fix Version/s 3.3.2 [ 12315108 ]
        Fix Version/s 3.4.0 [ 12314469 ]
        Hide
        Benjamin Reed added a comment -

        +1 nice catch!

        Show
        Benjamin Reed added a comment - +1 nice catch!
        Benjamin Reed made changes -
        Hadoop Flags [Reviewed]
        Hide
        Benjamin Reed added a comment -

        Committed revision 1022824.

        Show
        Benjamin Reed added a comment - Committed revision 1022824.
        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
        Flavio Junqueira added a comment -

        Ben forgot to close this issue.

        Show
        Flavio Junqueira added a comment - Ben forgot to close this issue.
        Flavio Junqueira made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        Patrick Hunt made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        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.)
        Flavio Junqueira made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Mahadev konar made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        15d 22h 22m 1 Jared Cantwell 14/Oct/10 23:03
        Patch Available Patch Available Resolved Resolved
        3d 18h 22m 1 Flavio Junqueira 18/Oct/10 17:26
        Resolved Resolved Reopened Reopened
        4m 54s 1 Patrick Hunt 18/Oct/10 17:30
        Reopened Reopened Resolved Resolved
        1h 59m 1 Flavio Junqueira 18/Oct/10 19:30
        Resolved Resolved Closed Closed
        401d 51m 1 Mahadev konar 23/Nov/11 19:21

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development