Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-4591

Clarify implementation and documentation of TarReader#mark

    XMLWordPrintableJSON

    Details

      Description

      There is a few peculiarities with that method:

      • The Javadoc "A bulk segment is reclaimable if it is in bulkRefs" is wrong. It should be "A bulk segment is reclaimable if it is not in bulkRefs".
      • (Why) is it necessary to iterate in reverse over the entries in tar file?
      • Why the extra check for bulk references in the else branch?
      • The condition !reclaim.remove(id) is always true as id can only be in reclaim it it had been added in the same iteration (as ids are unique). But this would have been in the if branch, contradicting us being in the else branch.

        Attachments

          Activity

            People

            • Assignee:
              mduerig Michael Dürig
              Reporter:
              mduerig Michael Dürig
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: