)]}' { "commit": "9829031ee9dd47600bfc1749d99f2737ac349d64", "tree": "22c4a15d4c72a79ec6be90599be5d7224fa3381d", "parents": [ "5890e16ec75053c4d82c1d596e415c2401e5d047" ], "author": { "name": "Marcin Czech", "email": "maczech@gmail.com", "time": "Mon Dec 13 18:17:41 2021 +0100" }, "committer": { "name": "Marcin Czech", "email": "maczech@gmail.com", "time": "Wed Dec 15 10:17:26 2021 +0000" }, "message": "Fix issue with skipping events from high-availability\n\nForwarderTask class keeps caller thread object to identify the source\nof the event. All events generated by `HTTP POST\n/plugins/high-availability/index/change/...` thread are skipped.\n\nThe issue is that all threads in HTTP thread pool are named `HTTP-*`\nwhen HTTP call is made thread is renamed to `HTTP POST\n/plugins/high-availability/index/change/...`. During the processing\nnew thread to send the message is created and current HTTP thread is\nreturned to the pool and it\u0027s name is set back to `HTTP-*`.\n\nThis means that if the code is not fast enough to check HTTP thread\nname before thread is returned to the pool it will get `HTTP-*`\ninstead of `HTTP POST /plugins/high-availability/index/change/...`\nand the event will be forwarded. To avoid this situation keep the\ncaller thread name instead of thread object.\n\nBug: Issue 15416\nChange-Id: I76f5921793cf844ad638f920127e8f2848a2615e\n", "tree_diff": [ { "type": "modify", "old_id": "329b5cb2847d26d2a90d052a86abaa91bd5d5a35", "old_mode": 33188, "old_path": "src/main/java/com/googlesource/gerrit/plugins/multisite/forwarder/ForwarderTask.java", "new_id": "c5706a50302b199c6d07b9ebdce12e26f5d1cb74", "new_mode": 33188, "new_path": "src/main/java/com/googlesource/gerrit/plugins/multisite/forwarder/ForwarderTask.java" }, { "type": "modify", "old_id": "49e6e1e106f75e6f0bded8e743897129a017d013", "old_mode": 33188, "old_path": "src/main/java/com/googlesource/gerrit/plugins/multisite/forwarder/broker/BrokerForwarder.java", "new_id": "56167db5ae894f3f2facd043d4215657ae34cb96", "new_mode": 33188, "new_path": "src/main/java/com/googlesource/gerrit/plugins/multisite/forwarder/broker/BrokerForwarder.java" } ] }