)]}'
{
  "commit": "66af3d8517f4ea6aeeb1b192cdb78dea24ecd3a2",
  "tree": "dfb98c3424d666c237bddbb795635870183fc188",
  "parents": [
    "2df725626c54fe94b98615f97034ae053b1b20a6"
  ],
  "author": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Tue Nov 10 17:38:40 2015 -0800"
  },
  "committer": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Wed Nov 11 12:16:51 2015 -0800"
  },
  "message": "Return list of reviewers as part of ChangeInfo\n\nChangeInfo now includes the list of reviewers when DETAILED_LABELS are\nrequested.\n\nSo far the list of reviewers could only indirectly be computed from\nthe returned approvals, but this relied on having a dummy 0 approval\nfor users that were added as reviewer, but that haven\u0027t voted yet.\n\nWhen notedb is enabled we no longer store dummy 0 approvals for\nreviewers, but we track the list of reviewers explicitly in notedb.\n\nWhen notedb was enabled the ChangeIT#addReviewerToClosedChange() test\nwas failing because it relied on the dummy 0 approvals.\n\nWith this change, reviewers are now included in ChangeInfo, adapt the\ntests to look at this field instead of computing the list of reviewers\nfrom the approvals.\n\nWhen notedb is enabled the PostReviewers REST endpoint records users\nas \u0027REVIEWER\u0027 in notedb. If notedb is disabled the PostReviewers REST\nendpoint behaves differently. In this case it adds dummy 0 approvals\nfor the users, but these translate into \u0027CC\u0027s when the ChangeInfo is\nconstructed. Hence the tests that check on reviewers must do different\nassertions depending on whether notedb is enabled or not.\n\nChange-Id: I5e7b99d85d97abc43ac17da44b7179f4ceb3e268\nSigned-off-by: Edwin Kempin \u003cekempin@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "8ae56e5d5918a7f7064563f4d671e1522a0a697f",
      "old_mode": 33188,
      "old_path": "Documentation/rest-api-changes.txt",
      "new_id": "46791583c93e78912ad60bf86370be8402a594fe",
      "new_mode": 33188,
      "new_path": "Documentation/rest-api-changes.txt"
    },
    {
      "type": "modify",
      "old_id": "92d7980d5a437608e7740fee64a86e7aa42197d4",
      "old_mode": 33188,
      "old_path": "gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java",
      "new_id": "159877461fd9ca45d88365e013d943f577b7f8ff",
      "new_mode": 33188,
      "new_path": "gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/change/ChangeIT.java"
    },
    {
      "type": "rename",
      "old_id": "b829a69f4bb9eafcd005553678bebac01a05cca1",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ReviewerState.java",
      "new_id": "a58c9591a9b1e5c25beb36bcd28bec90c4330236",
      "new_mode": 33188,
      "new_path": "gerrit-extension-api/src/main/java/com/google/gerrit/extensions/client/ReviewerState.java",
      "score": 63
    },
    {
      "type": "modify",
      "old_id": "cdfe0c6372c4a241a77df1d0847d5102e9d39ffa",
      "old_mode": 33188,
      "old_path": "gerrit-extension-api/src/main/java/com/google/gerrit/extensions/common/ChangeInfo.java",
      "new_id": "455243a86fe43b44044b71a295f6929192faa955",
      "new_mode": 33188,
      "new_path": "gerrit-extension-api/src/main/java/com/google/gerrit/extensions/common/ChangeInfo.java"
    },
    {
      "type": "modify",
      "old_id": "31058bc343dec202047e97a9dd439d431829d6db",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/ApprovalsUtil.java",
      "new_id": "7551d5f4e64ab84cfe8ffa13a18098544d1728f5",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/ApprovalsUtil.java"
    },
    {
      "type": "modify",
      "old_id": "733d7a26ea11cbfc5b859121bcfbe30dbce1f52e",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeJson.java",
      "new_id": "a02f0db7807099e344901d639f782ecd30d4a87f",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeJson.java"
    },
    {
      "type": "modify",
      "old_id": "c34fb4326c39226ef95bd84e117c6a0a89a64939",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java",
      "new_id": "efe31e3d5742a4f19598c7345ce81652ca468bd5",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java"
    },
    {
      "type": "modify",
      "old_id": "39f7bb46617cd87f03b226c041853c5b8b758c5c",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java",
      "new_id": "f2d0ffa4a4f74ce0d2f435cc88d2a02b88df99ce",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java"
    },
    {
      "type": "modify",
      "old_id": "05e0ceccc98fb44d89c2285e5dc7df901dc39386",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/git/MergeOp.java",
      "new_id": "a2d71fbfabf487599d2895e638a33c37bffe2bf3",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/git/MergeOp.java"
    },
    {
      "type": "modify",
      "old_id": "8948ce3f7cc33412f68d1811a35d2edbe0323ef4",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/mail/ChangeEmail.java",
      "new_id": "43d02fbecfc32ecd9622c84fa07fe31953e86a84",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/mail/ChangeEmail.java"
    },
    {
      "type": "modify",
      "old_id": "c6e59eb44d1f5800c422b49e3762be95349a20e4",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/mail/MailUtil.java",
      "new_id": "a3a048a854f9c7a13de4d1c7084260d9650fed91",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/mail/MailUtil.java"
    },
    {
      "type": "modify",
      "old_id": "7331954a44fccd624eb2304503870c3a7f8fe124",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeNotes.java",
      "new_id": "801f17219430263dc00f3b120d35a0c0f0d6962a",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeNotes.java"
    },
    {
      "type": "modify",
      "old_id": "fd4817f28c2c20c5f16fafe90cbf6bfff02e5ad1",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeNotesParser.java",
      "new_id": "b58548c9b2622e39f941be6668f7a8b3ba3882ff",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeNotesParser.java"
    },
    {
      "type": "modify",
      "old_id": "92a9856146ebc2237d7620d72d3d100aaf69073c",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeUpdate.java",
      "new_id": "1d8e5076911502d9f6a3563a0fa8a1faedb9400f",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeUpdate.java"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "3bf2135d1b1fac6fdbd8468887591271b6e5e500",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/notedb/ReviewerStateInternal.java"
    },
    {
      "type": "modify",
      "old_id": "1b7713bded61f2caee71914c61e57b57926836c2",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java",
      "new_id": "c06029b2aea10ee0c05d0c710e6349ace406a22b",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java"
    },
    {
      "type": "modify",
      "old_id": "e74df8fdcfc7cf6dcbaa147d9c2d756a88f6b2bb",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/test/java/com/google/gerrit/server/notedb/ChangeNotesTest.java",
      "new_id": "7debe94109c3927fcfd365b3127bdb9fe39ee548",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/test/java/com/google/gerrit/server/notedb/ChangeNotesTest.java"
    },
    {
      "type": "modify",
      "old_id": "c20690233f7271151e73fe0d998c261d8eef1710",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/test/java/com/google/gerrit/server/notedb/CommitMessageOutputTest.java",
      "new_id": "811cd8a34a54c539d364973cf5c4e6d5dbae9b48",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/test/java/com/google/gerrit/server/notedb/CommitMessageOutputTest.java"
    }
  ]
}