)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"1251818b7b7fa8ec28fbd303f39678e22a6666d0","unresolved":false,"context_lines":[{"line_number":9,"context_line":"All of these functions only have a single caller now, so move them to"},{"line_number":10,"context_line":"their calling site. Two functions are renamed to better fit their"},{"line_number":11,"context_line":"purpose but the only change in functionality here is that one function"},{"line_number":12,"context_line":"loses a memoize decorator, which isn\u0027t a serious issue since the work"},{"line_number":13,"context_line":"it\u0027s doing is minimal at best."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I48237f306fb901dec9ec805704c891fd3cbb12e3"},{"line_number":16,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_a3cf2227","line":13,"range":{"start_line":12,"start_character":33,"end_line":13,"end_character":29},"updated":"2019-09-27 14:01:36.000000000","message":"Is it? It looks like it might be doing ovo retrieval and deserialization from the database. That\u0027s expensive, even for a trivial object like S3ImageMapping. Else I imagine the memoization wouldn\u0027t have been put there in the first place.\n\n[Later] Um, it looks like you didn\u0027t actually remove memoization from anything. What am I missing?","commit_id":"085b5078810637a22d0dfdd99047e6edc6440e9e"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e1de6e305d38f241b58849d45503db148cd442b0","unresolved":false,"context_lines":[{"line_number":9,"context_line":"All of these functions only have a single caller now, so move them to"},{"line_number":10,"context_line":"their calling site. Two functions are renamed to better fit their"},{"line_number":11,"context_line":"purpose but the only change in functionality here is that one function"},{"line_number":12,"context_line":"loses a memoize decorator, which isn\u0027t a serious issue since the work"},{"line_number":13,"context_line":"it\u0027s doing is minimal at best."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I48237f306fb901dec9ec805704c891fd3cbb12e3"},{"line_number":16,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_152e69fa","line":13,"range":{"start_line":12,"start_character":33,"end_line":13,"end_character":29},"in_reply_to":"3fa7e38b_a3cf2227","updated":"2019-09-27 14:30:13.000000000","message":"Oh, weird. Guess I managed how to fix this at some point \\o/","commit_id":"085b5078810637a22d0dfdd99047e6edc6440e9e"}]}