Just adding a note that I tested this code with production data, and saw no noticeable performance issues when displaying the holds count for an item that had 600+ holds. I'm also noting that, due to the way the count is generated, you can have different counts for different copies on the same record. I don't think it's a problem, but I'm just making note of it because some users may expect the holds count to match the count they see in the bib record.
Just adding a note that I tested this code with production data, and saw no noticeable performance issues when displaying the holds count for an item that had 600+ holds. I'm also noting that, due to the way the count is generated, you can have different counts for different copies on the same record. I don't think it's a problem, but I'm just making note of it because some users may expect the holds count to match the count they see in the bib record.
The branch will need a release notes entry.