Jackrabbit Oak
  1. Jackrabbit Oak
  2. OAK-571

Limit usage of headRevId in sync collection to non-branches

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.6
    • Component/s: mongomk
    • Labels:
      None

      Description

      Currently the head revision in the sync collection is also updated for branch commits. This makes some commands more complicated (e.g. FetchHeadRevisionIdAction must check for branch commits). With the current implementation it may also happen that a commit is exposed, which is later marked as failed. This happens when the head revision is set to a higher value (because of a branch commit) than the current head revision of the trunk. Now it may happen that a commit with revision R to trunk with head-of-trunk < R < head-of-branch is exposed as valid commit for some time but may later marked failed.

      1. OAK-571.patch
        2 kB
        Mete Atamel
      2. OAK-571-2.patch
        2 kB
        Mete Atamel
      3. OAK-571-3.patch
        4 kB
        Marcel Reutegger
      4. OAK-571-4.patch
        7 kB
        Mete Atamel

        Issue Links

          Activity

          Marcel Reutegger created issue -
          Marcel Reutegger made changes -
          Field Original Value New Value
          Link This issue blocks OAK-560 [ OAK-560 ]
          Mete Atamel made changes -
          Attachment OAK-571.patch [ 12565771 ]
          Mete Atamel made changes -
          Attachment OAK-571-2.patch [ 12565920 ]
          Marcel Reutegger made changes -
          Attachment OAK-571-3.patch [ 12565947 ]
          Mete Atamel made changes -
          Attachment OAK-571-4.patch [ 12565952 ]
          Mete Atamel made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Jukka Zitting made changes -
          Fix Version/s 0.6 [ 12323297 ]
          Jukka Zitting made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Marcel Reutegger
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development