)]}'
{"nova/scheduler/client/report.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"589bc434238f78d3ffc53a38293b5bbfceaca1cc","unresolved":false,"context_lines":[{"line_number":384,"context_line":"               \"%(status_code)d: %(err_text)s.\")"},{"line_number":385,"context_line":"        query_traits \u003d [v for k, v in qs_params if k \u003d\u003d \u0027required\u0027]"},{"line_number":386,"context_line":"        query_aggs \u003d [v for k, v in qs_params if k \u003d\u003d \u0027member_of\u0027]"},{"line_number":387,"context_line":"        args[\u0027traits\u0027] \u003d query_traits and query_traits[0] or \u0027(none)\u0027"},{"line_number":388,"context_line":"        args[\u0027aggregates\u0027] \u003d \u0027,\u0027.join(query_aggs) or \u0027(none)\u0027"},{"line_number":389,"context_line":"        LOG.error(msg, args)"},{"line_number":390,"context_line":"        return None, None, None"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_e220d49d","line":387,"range":{"start_line":387,"start_character":25,"end_line":387,"end_character":69},"updated":"2018-05-14 16:17:33.000000000","message":"This is a bit obscure.  Would read better as\n\n args[\u0027traits\u0027] \u003d query_traits[0] if query_traits else \u0027(none)\u0027","commit_id":"ac114ecc630b9ad9e4fa67d42d07f850bec6e5ea"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"589bc434238f78d3ffc53a38293b5bbfceaca1cc","unresolved":false,"context_lines":[{"line_number":385,"context_line":"        query_traits \u003d [v for k, v in qs_params if k \u003d\u003d \u0027required\u0027]"},{"line_number":386,"context_line":"        query_aggs \u003d [v for k, v in qs_params if k \u003d\u003d \u0027member_of\u0027]"},{"line_number":387,"context_line":"        args[\u0027traits\u0027] \u003d query_traits and query_traits[0] or \u0027(none)\u0027"},{"line_number":388,"context_line":"        args[\u0027aggregates\u0027] \u003d \u0027,\u0027.join(query_aggs) or \u0027(none)\u0027"},{"line_number":389,"context_line":"        LOG.error(msg, args)"},{"line_number":390,"context_line":"        return None, None, None"},{"line_number":391,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_e25534fb","line":388,"range":{"start_line":388,"start_character":29,"end_line":388,"end_character":61},"updated":"2018-05-14 16:17:33.000000000","message":"To be clear, this will read as e.g.:\n\n in:agg1,agg2,in:agg3\n\nPerhaps a different delimiter than \u0027,\u0027 here would look better.","commit_id":"ac114ecc630b9ad9e4fa67d42d07f850bec6e5ea"}]}