)]}'
{
  "commit": "135d416f6732f311bb620a22f79bd097247235ee",
  "tree": "abd605f31b61bf47ce4353681ca006c313cb2759",
  "parents": [
    "a05c1d01b97e04427121bcc37a7b2c8e6f36c69c"
  ],
  "author": {
    "name": "Luca Milanesio",
    "email": "luca.milanesio@gmail.com",
    "time": "Sat Dec 09 00:17:05 2023 +0000"
  },
  "committer": {
    "name": "Luca Milanesio",
    "email": "luca.milanesio@gmail.com",
    "time": "Mon Dec 11 14:00:55 2023 +0000"
  },
  "message": "Do not take draft comments into account for change index up-to-date\n\nThe change index does not include draft comments.\nAlso, draft comments are not stored with the change notes\nand are user-specific, therefore do not make sense in the\noverall computation of the change latest update timestamp.\n\nPreviously, the status of the change latest ts was calculated\nwith the assumption that adding a draft comment would result\nin the change status to be modified and therefore needed\nto be reindexed up to the latest timestamp.\n\nIn multi-site the draft comments are not even replicated\nbecause of the lack of support for draft comments stream\nevents, added only from Gerrit v3.9.\n\nThe consequence was the creation and indefinite retry\nof the indexing events, which would have eventally clogged\nthe logs and the executors.\n\nTODO: This check needs to be reintroduced after the merging to\nstable-3.9, with the additional check of the draft comment stream\nevents being enabled or not.\n\nBug: Issue 40014838\nChange-Id: I2f7e63bd3a97c4463fb4f5417770cbb429026c2b\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "4ef5b7fdac8799ac88f41736a4d888c38f4022d9",
      "old_mode": 33188,
      "old_path": "src/main/java/com/googlesource/gerrit/plugins/multisite/index/ChangeCheckerImpl.java",
      "new_id": "6e8fa152b7dfc38e8171773af66db5020c9024c3",
      "new_mode": 33188,
      "new_path": "src/main/java/com/googlesource/gerrit/plugins/multisite/index/ChangeCheckerImpl.java"
    }
  ]
}
