)]}'
{
  "commit": "70e28b4ae72e6200424beeaa5c9687473a1fcd6e",
  "tree": "744904aeef5a1ff55ca20e238b0edafee0af56b5",
  "parents": [
    "a904bf23edf4cb7d4f65f40626c78dcc169c2bf6"
  ],
  "author": {
    "name": "Alice Kober-Sotzek",
    "email": "aliceks@google.com",
    "time": "Thu Aug 20 12:00:54 2020 +0200"
  },
  "committer": {
    "name": "Alice Kober-Sotzek",
    "email": "aliceks@google.com",
    "time": "Thu Aug 20 12:00:54 2020 +0200"
  },
  "message": "ValidatorConfig: Avoid \u0027stream::iterator\u0027\n\nThe method reference stream::iterator is considered unsafe as it\u0027s\nmapped to the Iterable interface which can be iterated multiple times\nwhereas a stream may only be iterated once.\n\nI268068f84 did a similar cleanup in core Gerrit. This change improves\nthe only usage of stream::iterator in the uploadvalidator plugin.\n\nIn the specific case of ValidatorConfig, we can safely switch to a\nmanifestation of an ImmutableList as ValidatorConfig#groupUUID returns\nonly non-null values and conf.getStringList(\"skipGroup\") already\nreads/manifests all values. Hence, there shouldn\u0027t be any negative\nside-effects when switching to an ImmutableList here.\n\nChange-Id: Ib81ef3b5e0e6e254d4e3189de49abfa37415771d\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "4aae4d6adb2adfe42215db9f1c96891e0d283c63",
      "old_mode": 33188,
      "old_path": "src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/ValidatorConfig.java",
      "new_id": "aea74a69042161f110543f0df85c4935d9afd59b",
      "new_mode": 33188,
      "new_path": "src/main/java/com/googlesource/gerrit/plugins/uploadvalidator/ValidatorConfig.java"
    }
  ]
}