)]}'
{"mistral/db/v2/sqlalchemy/api.py":[{"author":{"_account_id":19206,"name":"Dawid Deja","email":"dawdej@gmail.com","username":"ddeja"},"change_message_id":"cdbebe0285b979318f6239d0c07a38494665ba5d","unresolved":false,"context_lines":[{"line_number":815,"context_line":""},{"line_number":816,"context_line":""},{"line_number":817,"context_line":"def get_incomplete_task_executions_count(**kwargs):"},{"line_number":818,"context_line":"    query \u003d b.model_query(models.TaskExecution)"},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"    query \u003d query.filter_by(**kwargs)"},{"line_number":821,"context_line":""},{"line_number":822,"context_line":"    query \u003d query.filter("},{"line_number":823,"context_line":"        sa.or_("},{"line_number":824,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027IDLE\u0027,"},{"line_number":825,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027RUNNING\u0027,"},{"line_number":826,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027WAITING\u0027,"},{"line_number":827,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027DELAYED\u0027"},{"line_number":828,"context_line":"        )"},{"line_number":829,"context_line":"    )"},{"line_number":830,"context_line":""},{"line_number":831,"context_line":"    return query.count()"},{"line_number":832,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_c3365d02","line":829,"range":{"start_line":818,"start_character":0,"end_line":829,"end_character":5},"updated":"2016-09-13 09:31:39.000000000","message":"This lines are redundant with lines 801-812. Can we extract it to separate private method and use it in both places? It would be easier to maintain.","commit_id":"e654d67f640f4e2366469492601f46cd15e22e0d"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"abff316456e23774e38721af76130036762c59a4","unresolved":false,"context_lines":[{"line_number":815,"context_line":""},{"line_number":816,"context_line":""},{"line_number":817,"context_line":"def get_incomplete_task_executions_count(**kwargs):"},{"line_number":818,"context_line":"    query \u003d b.model_query(models.TaskExecution)"},{"line_number":819,"context_line":""},{"line_number":820,"context_line":"    query \u003d query.filter_by(**kwargs)"},{"line_number":821,"context_line":""},{"line_number":822,"context_line":"    query \u003d query.filter("},{"line_number":823,"context_line":"        sa.or_("},{"line_number":824,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027IDLE\u0027,"},{"line_number":825,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027RUNNING\u0027,"},{"line_number":826,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027WAITING\u0027,"},{"line_number":827,"context_line":"            models.TaskExecution.state \u003d\u003d \u0027DELAYED\u0027"},{"line_number":828,"context_line":"        )"},{"line_number":829,"context_line":"    )"},{"line_number":830,"context_line":""},{"line_number":831,"context_line":"    return query.count()"},{"line_number":832,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_dee390d6","line":829,"range":{"start_line":818,"start_character":0,"end_line":829,"end_character":5},"in_reply_to":"9a89bdaa_c3365d02","updated":"2016-09-13 09:36:08.000000000","message":"Yes, sure, good idea.","commit_id":"e654d67f640f4e2366469492601f46cd15e22e0d"}]}