)]}'
{"openstack_health/api.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d43015d03f8afb29de3208072d9c2aaf93ec619d","unresolved":false,"context_lines":[{"line_number":499,"context_line":""},{"line_number":500,"context_line":"    def _populate_bug_dict(change_num, patch_num, short_uuid, run):"},{"line_number":501,"context_line":"        bug_dict[run] \u003d classifier.classify(change_num, patch_num,"},{"line_number":502,"context_line":"                                            short_uuid, recent\u003dTrue)"},{"line_number":503,"context_line":""},{"line_number":504,"context_line":"    @region.cache_on_arguments()"},{"line_number":505,"context_line":"    def _get_recent(status):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_df169c57","side":"PARENT","line":502,"range":{"start_line":502,"start_character":55,"end_line":502,"end_character":67},"updated":"2016-08-18 20:50:36.000000000","message":"I had a concern about this removal recent, but that is fine for us because of the following conversation:\n\n 13:45 (oomichi) mtreinish: oh, cool. One question is that _populate_bug_dict() called classify() with recent\u003dTrue, but the new one doesn\u0027t\n 13:45 (oomichi) mtreinish: is it fine for us?\n 13:46 (mtreinish) oomichi: for caching reasons. The recent\u003dTrue means limit the search results to 1 day\n 13:46 (mtreinish) which only applies to the es queries on the home page\n 13:46 (mtreinish) that\u0027s an optimization for performance of the query, but I figured increasing cache hits was more important than that\n\nIt is nice to see the performance result more later after merging this.","commit_id":"a5d7712cd9ee7e81cd5dbee7620f11f95ea2c7bb"}]}