-
Stan Hu authored
In a project with JIRA activated, `ProcessCommitWorker` attempts to add a comment to a JIRA issue if that issue is mentioned in a commit. However, the JIRA integration would attempt to retrieve all tags and branches and pick the first matching ref given a commit OID. The problem with that approach is that after each push, the list of all branches and tags are expired and could take a while to gather. Since multiple `ProcessCommitWorker` jobs can be running at the same time, this can lead to high I/O on Gitaly nodes since multiple `ProcessCommitWorker` jobs can run at the same time. We observe that we don't really need to build the entire ref list; we can just ask Gitaly for a single matching ref for the given OID with the newly-created `FindRefsByOID` RPC introduced in gitaly!3947. Relates to #343035 Changelog: performance
1993feb5
Code owners
:
Zack Cuddy, Florie Guibert, David O'Regan, Ezekiel Kigbo, Phil Hughes, Martin Wortschack, Natalia Tepluhina, Mike Greiling, Tim Zallmann, Kushal Pandya, Paul Slaughter, Enrique Alcántara, Illya Klymov, Andrew Fontaine, Denys Mishunov, Jose Ivan Vargas, Olena Horal-Koretska, Miguel Rincon, Savas Vedova, Jacques Erasmus, Simon Knox, Scott Hampton, Peter Hegman, Mark Florian, Jiaan Louw, Robert Hunt, Nicolò Maria Mezzopera, Vitaly Slobodin, Brandon Labuschagne, Michael Lunøe, Frédéric Caplette, David Pisek, Alexander Turinske, Jannik Lehmann, Anna Vovchenko, Michele Bursi, Marin Jankovski, Amy Phillips, Alessio Caiazza, John Skarbek, Ahmad Tolba, Graeme Gillies, Jenny Kim, Matt Field, Mayra Cabrera, Reuben Pereira, GitLab Release Tools Bot, James Lopez, Peter Leitzen, Huzaifa Iftikhar, Sean McGivern, Douglas Barbosa Alexandre, Stan Hu, Grzegorz Bizon, Rémy Coutable, Kamil Trzciński, Bob Van Landuyt, Alexandru Croitor, Jan Provaznik, Thong Kuah, Heinrich Lee Yu, Imre Farkas, charlie ablett, Dylan Griffith, Albert Salim, Shinya Maeda, Jarka Košanová, James Fargher, Sincheol (David) Kim, Igor Drozdov, Mikołaj Wawrzyniak, Matthias Käppler, Vitali Tatarintev, Patrick Bajao, Fabio Pitino, Lin Jen-Shin, David Fernandez, Mark Chao, Kerri Miller, Alper Akgun, Terri Chu, Gabriel Mazetto, Aleksei Lipniagov, Sean Arnold, Etienne Baqué, Arturo Herrero, Alex Kalderimis, Nikola Milojevic, Dmitry Gruzd, Doug Stull, Alex Pooley, Robert May, Vasilii Iakliushin, Andy Soiron, Tetiana Chupryna, Allison Browne, Pavel Shutsin, Furkan Ayhan, Marius Bobin, Marc Shaw, Ethan Urie, Michael Kozono, Ash McKenzie, Luke Duncalfe, Max Woolf, George Koltsov, Allen Cook, Steve Abrams, Tiger Watson, Krasimir Angelov, Alex Ives, João Alexandre Cunha, Adam Hegyi, Simon Tomlinson, Diogo Frazão, Chloe Liu, Tiffany Rea, Ramya Authappan, Anastasia McDonald, Andrejs Cunskis, Dan Davison, Mark Lapierre, Sanad Liaquat, Hordur Freyr Yngvason, Cheryl Li, Marcel Amirault, Laura Montemayor, Jacob Vosmaer, and Kyle Wiebers