)]}'
{
  "commit": "5a97145bef520ed025b1fccafbd8e0e0331c9fe9",
  "tree": "21db09cb0f015c1ceac56a6b3ac1e9e382038ae9",
  "parents": [
    "00f338840c6fefbdc1d2661faab3ed525c877d83"
  ],
  "author": {
    "name": "Edwin Kempin",
    "email": "edwin.kempin@sap.com",
    "time": "Wed Jan 08 14:58:05 2014 +0100"
  },
  "committer": {
    "name": "Edwin Kempin",
    "email": "edwin.kempin@sap.com",
    "time": "Fri Jan 10 16:37:41 2014 +0100"
  },
  "message": "Update mergeability flag during reindex\n\nBefore commit f8be8fdca the mergeability flag on changes was only\nupdated on request and for most of the changes the persisted\nmergeability flag was outdated. Commit f8be8fdca makes sure to keep the\nmergeability flag updated whenever the destination branch is updated,\nbut for old changes the mergeability flag may still be outdated. Since\nsome Gerrit functionality is relying on the mergeability flag\nbeing up to date there must be a way to update the mergeability flag\nfor all old changes. One possibility would be to do this as a schema\nmigration but there may be situations where it\u0027s needed to update the\nmergeability flag on demand (e.g. if during a Gerrit downtime branches\nare updated due to some administration tasks). This is why it\u0027s\nbetter to update the mergeability flag in the reindexer when the\nchanges are reindexed. Also the schema migration is done during the\ninit phase where a lot of classes that are needed by the\nMergeabilityChecker are not bound. The reindexer on the other hand\nalready has a lot of the needed Guice bindings in place.\n\nAn outdated mergeability flag is especially bad since this information\nis shown in the change tables and user dashboards (see commit\n56f39eb0a0). Users will be very confused by seeing wrong (outdated)\ninformation here which is why it is important to get this fixed before\nthe next release.\n\nBug: issue 2322\nChange-Id: If85839cedcfed25e872b264149d13e98b10ba835\nSigned-off-by: Edwin Kempin \u003cedwin.kempin@sap.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "5396014e1e5b9b1788de6ea956b08996fe7bb413",
      "old_mode": 33188,
      "old_path": "gerrit-pgm/src/main/java/com/google/gerrit/pgm/Reindex.java",
      "new_id": "afe62316881070963fcbfe5a34930346240ef5bc",
      "new_mode": 33188,
      "new_path": "gerrit-pgm/src/main/java/com/google/gerrit/pgm/Reindex.java"
    },
    {
      "type": "modify",
      "old_id": "bf9f65d510a6db8c917950741e752a7eb8ef7955",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/MergeabilityChecker.java",
      "new_id": "bb4027a488f3654d8700b5d97a19f96553913b83",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/change/MergeabilityChecker.java"
    },
    {
      "type": "modify",
      "old_id": "c5327b4d1f44d301bf399827a6e52ead4c518dc3",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/config/GerritGlobalModule.java",
      "new_id": "309b1516530da7061e5130427d396318de3f52ac",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/config/GerritGlobalModule.java"
    },
    {
      "type": "modify",
      "old_id": "c9465102b42beacdcad28ea916eb7f94e743ffa6",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/index/ChangeBatchIndexer.java",
      "new_id": "0334715ccc5fcff970120e7c0b055ab8bd771490",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/index/ChangeBatchIndexer.java"
    },
    {
      "type": "modify",
      "old_id": "a4946ebc3a9e53b9257073470107ad22829ecc3c",
      "old_mode": 33188,
      "old_path": "gerrit-server/src/main/java/com/google/gerrit/server/project/AccessControlModule.java",
      "new_id": "7215d18177384b476b38a9d332d5d7a5d4657d0c",
      "new_mode": 33188,
      "new_path": "gerrit-server/src/main/java/com/google/gerrit/server/project/AccessControlModule.java"
    }
  ]
}