Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3.0
    • Component/s: contrib-bookkeeper
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      fix the overview section in the bookkeeper documentation to introduce the programmer/admin to bookkeeper before giving the details.

      1. bk-overview.jpg
        121 kB
        Flavio Junqueira
      2. ZOOKEEPER-607.patch
        16 kB
        Flavio Junqueira
      3. ZOOKEEPER-607.patch
        18 kB
        Flavio Junqueira
      4. ZOOKEEPER-607.patch
        19 kB
        Flavio Junqueira
      5. ZOOKEEPER-607.patch
        17 kB
        Flavio Junqueira
      6. ZOOKEEPER-607.patch
        17 kB
        Flavio Junqueira

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in ZooKeeper-trunk #701 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/701/)

          Show
          Hudson added a comment - Integrated in ZooKeeper-trunk #701 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/701/ )
          Hide
          Mahadev konar added a comment -

          +1 the patch looks good.

          I just committed this flavio. really good to have this!!!

          thanks!

          Show
          Mahadev konar added a comment - +1 the patch looks good. I just committed this flavio. really good to have this!!! thanks!
          Hide
          Flavio Junqueira added a comment -

          I have simplified a bit the discussion on closing out ledgers. It was too much detail for an overview.

          Show
          Flavio Junqueira added a comment - I have simplified a bit the discussion on closing out ledgers. It was too much detail for an overview.
          Hide
          Flavio Junqueira added a comment -

          Please don't forget to include the figure.

          Show
          Flavio Junqueira added a comment - Please don't forget to include the figure.
          Hide
          Benjamin Reed added a comment -

          +1 good job flavio.

          Show
          Benjamin Reed added a comment - +1 good job flavio.
          Hide
          Flavio Junqueira added a comment -

          Overview figure.

          Show
          Flavio Junqueira added a comment - Overview figure.
          Hide
          Flavio Junqueira added a comment -

          Another preliminary patch addressing Ben's comment and introducing a figure. I'm uploading the figure separately, and it has to go to the resources/images folder to generate the docs. I also want to check that table of entry fields is consistent with the latest version of the system.

          Show
          Flavio Junqueira added a comment - Another preliminary patch addressing Ben's comment and introducing a figure. I'm uploading the figure separately, and it has to go to the resources/images folder to generate the docs. I also want to check that table of entry fields is consistent with the latest version of the system.
          Hide
          Mahadev konar added a comment -

          will be good to get this in 3.3

          Show
          Mahadev konar added a comment - will be good to get this in 3.3
          Hide
          Benjamin Reed added a comment -

          it looks good flavio just two suggestions:

          1) i think it is too much information when you start talking about the naming convention for the znodes in zookeeper.
          2) the motivation is good, but i don't think we really need to discussion the alternatives for hdfs.

          Show
          Benjamin Reed added a comment - it looks good flavio just two suggestions: 1) i think it is too much information when you start talking about the naming convention for the znodes in zookeeper. 2) the motivation is good, but i don't think we really need to discussion the alternatives for hdfs.
          Hide
          Flavio Junqueira added a comment -

          A preliminary patch that compiles (at least for me).

          Show
          Flavio Junqueira added a comment - A preliminary patch that compiles (at least for me).
          Hide
          Flavio Junqueira added a comment -

          Attaching preliminary patch.

          Show
          Flavio Junqueira added a comment - Attaching preliminary patch.

            People

            • Assignee:
              Flavio Junqueira
              Reporter:
              Benjamin Reed
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development