)]}'
{
  "commit": "f6928f57362d952b18a99d89d4778ca6ad75e84d",
  "tree": "5f6e58fcbb8c3bef3749bf99a8494d9181cf2803",
  "parents": [
    "2fd050c5674a08f1a5ccbe6b6cfc922ce6144c4a"
  ],
  "author": {
    "name": "Martin Fick",
    "email": "quic_mfick@quicinc.com",
    "time": "Wed Mar 08 08:34:38 2023 -0700"
  },
  "committer": {
    "name": "Luca Milanesio",
    "email": "luca.milanesio@gmail.com",
    "time": "Wed Jun 07 19:08:39 2023 -0400"
  },
  "message": "Revert \"RefDirectory: Throw exception if CAS of packed ref list fails\"\n\nThis reverts commit 9c33f7364d41956240818ba12d8b79d5ea846162.\n\nReason for revert: This change was based on the false claim that the\npackedrefs file lock is held while the CAS is being done, but it is\nactually released before the CAS (the in memory lock is still held,\nhowever that does not prevent external actors from updating the\npackedrefs files and then another thread from subsequently re-reading it\nand updating the in memory packedRefList). Although reverting this\nchange can cause the CAS to fail, it should not actually matter since\nthe failure would indicate that another thread has already updated the\nin memory packedRefList to either the same version this thread was\ntrying to update it too, or to a more recent version. Either way,\nfailing the CAS is then appropriate and should not be problematic.\n\nAlthough this change reverts the code in the RefDirectory class, it\nkeeps the \"improvements\" to the test so that it continues to pass\nreliably. The reason for the quotes around the word \"improvements\" is\nbecause I believe the test alteration actually dramatically changes the\nintent of the test, and that the original intent of the test is\nuntestable with the GC and RefDirectory classes as is.\n\nBug: 582044\nChange-Id: I3acee7527bb542996dcdfaddfb2bdb45ec444db5\nSigned-off-by: Martin Fick \u003cquic_mfick@quicinc.com\u003e\n(cherry picked from commit c5617711a1b4d5d0807cc7eed702b78d114d46b3)\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "b46ffe3670532ca7e035ebcf488b3ec94b79acb7",
      "old_mode": 33188,
      "old_path": "org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/file/RefDirectory.java",
      "new_id": "fbf0ae9cfa94aa120679d5d7c00f485182d3d52d",
      "new_mode": 33188,
      "new_path": "org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/file/RefDirectory.java"
    }
  ]
}
