)]}'
{
  "commit": "d269abb77be5d19e04df34e08d89ac1d643236a8",
  "tree": "cbc9094f1843b1a56b8f8ac90c375cca45c34d47",
  "parents": [
    "bca9e7704f82284312998687c16a78252b0f9939"
  ],
  "author": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Thu Mar 04 14:51:26 2021 +0100"
  },
  "committer": {
    "name": "Edwin Kempin",
    "email": "ekempin@google.com",
    "time": "Fri Mar 05 11:50:08 2021 +0100"
  },
  "message": "Compute only as many owned paths as needed, not all\n\nWhen a code owner approval is applied and when a code owner is added as\na reviewer we post a change message that lists the owned paths. To avoid\nthat the change messages get too large there is a configurable limit for\nthe maximum number of paths that should be shown in change messages. If\na user owns more paths the change message includes \"(X more files)\" to\nlet the user know about this. For being able to post this message we are\ncomputing all paths of the change that are owned by the user.\n\nComputing owned path is rather expensive and if a change touches many\nfiles this increases the latency of the post review call. To improve the\nlatency we are now limiting the computation of the owned paths to the\nnumber of paths that we are going to include into the change message, +1\nto know if there are more owned files. This means that the change\nmessage can no longer say how many more owned files there are, but\nimproving the latency seems more important than keeping this number\nincluded. If there are more owned files we will now only say \"(more\nfiles)\" to let the user know about this.\n\nSigned-off-by: Edwin Kempin \u003cekempin@google.com\u003e\nChange-Id: Id33722dc09144710758db5c84de4772656b9a4b5\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "8361d9b616ce9c11974551813abb4c5069f4ce11",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheck.java",
      "new_id": "1c5e87f9c4b96084810c3d140fc5946a7db6cc27",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/backend/CodeOwnerApprovalCheck.java"
    },
    {
      "type": "modify",
      "old_id": "9352e9751c563b1ac33fc25f2d21cc316582901e",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnAddReviewer.java",
      "new_id": "e6a55eeb6a35e409fb0249fc5fd3a4b8d4f6f291",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnAddReviewer.java"
    },
    {
      "type": "modify",
      "old_id": "101cedd65046f47bc419cc9622500e5f5c471483",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/backend/OnCodeOwnerApproval.java",
      "new_id": "e9e2c39d12b0205032740c3ee76dd065d305b2fd",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/backend/OnCodeOwnerApproval.java"
    },
    {
      "type": "modify",
      "old_id": "305b31aad77bdf361adc2460f2b95eb7e6e038fe",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/plugins/codeowners/restapi/GetOwnedPaths.java",
      "new_id": "0430f949136097f9f49dbb0dddb16a2d53fd1bad",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/plugins/codeowners/restapi/GetOwnedPaths.java"
    },
    {
      "type": "modify",
      "old_id": "576d2fe556d1d3e3ca4b18d23761016d10fd8265",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java",
      "new_id": "91c40db6256a87f4e3bd8b612caf33646e72d358",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/plugins/codeowners/acceptance/api/CodeOwnersOnAddReviewerIT.java"
    },
    {
      "type": "modify",
      "old_id": "971f8e9ab7455059b19c30bfb7c38c13fb93cb8b",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/plugins/codeowners/acceptance/api/OnCodeOwnerApprovalIT.java",
      "new_id": "8a8b9c41c0ed92fbeab3cd8a0f013d9eff8d2a1e",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/plugins/codeowners/acceptance/api/OnCodeOwnerApprovalIT.java"
    }
  ]
}
