)]}'
{
  "commit": "f9eadd05efa82e91c97f3fbe270392533dc3fd41",
  "tree": "b674ec639c21a45ad53502e49191b020b76eda8b",
  "parents": [
    "a90124b1dd503d54ec19db223535ce10342818d0"
  ],
  "author": {
    "name": "Saša Živkov",
    "email": "sasa.zivkov@sap.com",
    "time": "Thu Mar 10 17:40:50 2016 +0100"
  },
  "committer": {
    "name": "Saša Živkov",
    "email": "zivkov@gmail.com",
    "time": "Fri Mar 11 14:42:14 2016 +0000"
  },
  "message": "Instead of deleting patch-set-approval set vote to zero\n\nProblem: some reviewers get removed from change after it gets submitted.\n\nOn submitting a change we used to delete those patch-set-approvals where\nthe reviewer didn\u0027t have permission to vote in that label or the label\ndidn\u0027t exist any more.\n\nOn the other side, when a reviewer is added to a change this is recorded\nas patch-set-approval on some label, with zero vote. If the chosen label\nhappens to be one where the reviewer doesn\u0027t have permission to vote,\nthis patch-set-approval will be deleted on submit and the reviewer will\nbe removed from the change.\n\nEven if the added reviewer did have permission to vote in the chosen label,\nthe label could get deleted before the change is submitted. Again, the\nreviewer will be removed from the change.\n\nInstead of deleting patch-set-approvals set the vote to zero so that we\ndon\u0027t accidentally remove reviewers.\n\nNOTE: this issue doesn\u0027t occur when using NoteDb. In NoteDb we keep\ntrack of reviewers explicitly and this is why for NoteDb we still want\nto delete these approvals.\n\nChange-Id: I4d4f8d7b55c12a0484257192e7d0ed8d2f71ebcc\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "bdf9f052122dcbbbcdbf85861cf03720e48bd936",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/git/strategy/SubmitStrategyOp.java",
      "new_id": "b32d668b770dce19d8ea0d17e3f6572f445163db",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/git/strategy/SubmitStrategyOp.java"
    }
  ]
}