Bookkeeper
  1. Bookkeeper
  2. BOOKKEEPER-188

Garbage collection code is in the wrong place

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1.0
    • Component/s: None
    • Labels:
      None

      Description

      There's a lot of garbage collection code in the wrong place in EntryLogger. extractMetaFromEntryLogs, and EntryLogMetadata are only every used in the GC Thread. So they should move there.

        Activity

        Hide
        Ivan Kelly added a comment -

        This is a very straight forward refactor. The one functional change is that extractMetaFromEntryLogs now call extractMetaFromEntryLog rather than having duplicate logic in each.

        Show
        Ivan Kelly added a comment - This is a very straight forward refactor. The one functional change is that extractMetaFromEntryLogs now call extractMetaFromEntryLog rather than having duplicate logic in each.
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/4414/
        -----------------------------------------------------------

        Review request for bookkeeper.

        Summary
        -------

        This is a very straight forward refactor. The one functional change is that extractMetaFromEntryLogs now call extractMetaFromEntryLog rather than having duplicate logic in each.

        This addresses bug BOOKKEEPER-188.
        https://issues.apache.org/jira/browse/BOOKKEEPER-188

        Diffs


        bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java 8da372e
        bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/GarbageCollectorThread.java 23f02bc
        bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/EntryLogTest.java 96df89a

        Diff: https://reviews.apache.org/r/4414/diff

        Testing
        -------

        Thanks,

        Ivan

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4414/ ----------------------------------------------------------- Review request for bookkeeper. Summary ------- This is a very straight forward refactor. The one functional change is that extractMetaFromEntryLogs now call extractMetaFromEntryLog rather than having duplicate logic in each. This addresses bug BOOKKEEPER-188 . https://issues.apache.org/jira/browse/BOOKKEEPER-188 Diffs bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java 8da372e bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/GarbageCollectorThread.java 23f02bc bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/EntryLogTest.java 96df89a Diff: https://reviews.apache.org/r/4414/diff Testing ------- Thanks, Ivan
        Hide
        Sijie Guo added a comment -

        +1 for the patch, will commit it.

        Show
        Sijie Guo added a comment - +1 for the patch, will commit it.
        Hide
        Sijie Guo added a comment -

        committed as r1303286. thanks Ivan.

        Show
        Sijie Guo added a comment - committed as r1303286. thanks Ivan.
        Hide
        Hudson added a comment -

        Integrated in bookkeeper-trunk #420 (See https://builds.apache.org/job/bookkeeper-trunk/420/)
        BOOKKEEPER-188: Garbage collection code is in the wrong place (ivank via sijie) (Revision 1303286)

        Result = ABORTED
        sijie :
        Files :

        • /zookeeper/bookkeeper/trunk/CHANGES.txt
        • /zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java
        • /zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/GarbageCollectorThread.java
        • /zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/EntryLogTest.java
        Show
        Hudson added a comment - Integrated in bookkeeper-trunk #420 (See https://builds.apache.org/job/bookkeeper-trunk/420/ ) BOOKKEEPER-188 : Garbage collection code is in the wrong place (ivank via sijie) (Revision 1303286) Result = ABORTED sijie : Files : /zookeeper/bookkeeper/trunk/CHANGES.txt /zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java /zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/GarbageCollectorThread.java /zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/EntryLogTest.java

          People

          • Assignee:
            Ivan Kelly
            Reporter:
            Ivan Kelly
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development