Uploaded image for project: 'Bookkeeper'
  1. Bookkeeper
  2. BOOKKEEPER-188

Garbage collection code is in the wrong place

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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
        ikelly 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
        ikelly 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 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 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
        hustlmsp Sijie Guo added a comment -

        +1 for the patch, will commit it.

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

        committed as r1303286. thanks Ivan.

        Show
        hustlmsp Sijie Guo added a comment - committed as r1303286. thanks Ivan.
        Hide
        hudson 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 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:
            ikelly Ivan Kelly
            Reporter:
            ikelly Ivan Kelly
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development