)]}'
{
  "commit": "48ff71492703f04cd636682022b83ceb56cd88a7",
  "tree": "c94f369d283badc5fc3ae1fff40e53c502be9e0e",
  "parents": [
    "0980d3303642a75491fe340674a294b8142ecfc0"
  ],
  "author": {
    "name": "Gal Paikin",
    "email": "paiking@google.com",
    "time": "Mon Oct 18 17:23:45 2021 +0200"
  },
  "committer": {
    "name": "Gal Paikin",
    "email": "paiking@google.com",
    "time": "Tue Oct 19 13:17:07 2021 +0200"
  },
  "message": "Fix bug about persisting copied votes on submit\n\nThis bug is happening because when submitting a change, on some submit\nstrategies, we create a new patch-set which causes us to persist copied\nvotes. When persisting copied votes with ListOfFilesUnchanged copy\ncondition, we need to fetch the commit. This fails for non-existent\npatch-sets (we are just now creating it).\n\nInstead of fixing the above, we do the following:\n\nWe ensure we don\u0027t call this code at all for patch-sets created during\nsubmit. This is unnecessary since during submit we anyway store\napprovals (and they don\u0027t need to be \"copied approvals\"). It\u0027s best to\nmake sure we don\u0027t copy approvals twice, even though that bug never\nsurfaced (luck).\n\nChange-Id: Ibc61b390814ec12ed2bdd1b8dd199187c14cbea4\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "f0939584d9a833b68c9aaadd80677063a4ea924c",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/server/change/PatchSetInserter.java",
      "new_id": "209901d65c3cf493790f685d3c48f55d8f44094e",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/server/change/PatchSetInserter.java"
    },
    {
      "type": "modify",
      "old_id": "49524643e399fa12e7d04f9cffef92c4e3b4c85b",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/server/change/RebaseChangeOp.java",
      "new_id": "3e67cca3306cabffe14f7e8acd97d6ec84908b8e",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/server/change/RebaseChangeOp.java"
    },
    {
      "type": "modify",
      "old_id": "cc3b75dbc3fc9bb48ef31651045984933f6ebcb1",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/server/submit/RebaseSubmitStrategy.java",
      "new_id": "355d25fd226adec2de92bfcabd2d27cda0c1b79a",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/server/submit/RebaseSubmitStrategy.java"
    },
    {
      "type": "modify",
      "old_id": "fff3cb6c29a33d13cd324013835818f46216269b",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/acceptance/rest/change/SubmitByCherryPickIT.java",
      "new_id": "b3592e3d20554556485e6bba307d82f920d76255",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/acceptance/rest/change/SubmitByCherryPickIT.java"
    },
    {
      "type": "modify",
      "old_id": "eeeac2a4091d8f63aae548a61f39d448d3436205",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/acceptance/rest/change/SubmitByRebaseAlwaysIT.java",
      "new_id": "aa938157f15db2777e10b5d92c4c8aad32e69650",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/acceptance/rest/change/SubmitByRebaseAlwaysIT.java"
    }
  ]
}
