)]}'
{"watcher/decision_engine/strategy/context/default.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"528fc64c85e22710d49649acfe18a44835f3794e","unresolved":false,"context_lines":[{"line_number":60,"context_line":"        if audit_template.strategy_id:"},{"line_number":61,"context_line":"            strategy \u003d objects.Strategy.get_by_id(request_context,"},{"line_number":62,"context_line":"                                                  audit_template.strategy_id)"},{"line_number":63,"context_line":"            if strategy:"},{"line_number":64,"context_line":"                strategy_name \u003d strategy.name"},{"line_number":65,"context_line":"            else:"},{"line_number":66,"context_line":"                raise exception.StrategyNotFound(strategy\u003daudit_template."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_c3f8834c","line":63,"range":{"start_line":63,"start_character":12,"end_line":63,"end_character":24},"updated":"2016-06-09 16:18:09.000000000","message":"if objects.Strategy.get_by_id() does not return a strategy, it means that it raised an exception instead. So at this stage, this if statement seems unnecessary to me.","commit_id":"d8f6da6ec758a388ebfd40584bbe32987578c20a"},{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"0a27cf8282a0a4aa1e767755fcd531bd8fac68c7","unresolved":false,"context_lines":[{"line_number":60,"context_line":"        if audit_template.strategy_id:"},{"line_number":61,"context_line":"            strategy \u003d objects.Strategy.get_by_id(request_context,"},{"line_number":62,"context_line":"                                                  audit_template.strategy_id)"},{"line_number":63,"context_line":"            if strategy:"},{"line_number":64,"context_line":"                strategy_name \u003d strategy.name"},{"line_number":65,"context_line":"            else:"},{"line_number":66,"context_line":"                raise exception.StrategyNotFound(strategy\u003daudit_template."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_602fe1f6","line":63,"range":{"start_line":63,"start_character":12,"end_line":63,"end_character":24},"in_reply_to":"7aa08908_c3f8834c","updated":"2016-06-10 07:18:54.000000000","message":"you are right.","commit_id":"d8f6da6ec758a388ebfd40584bbe32987578c20a"}],"watcher/tests/decision_engine/strategy/context/test_strategy_context.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"528fc64c85e22710d49649acfe18a44835f3794e","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from watcher.common import utils"},{"line_number":19,"context_line":"from watcher.decision_engine.solution import default"},{"line_number":20,"context_line":"from watcher.decision_engine.strategy.context.default import \\"},{"line_number":21,"context_line":"    DefaultStrategyContext"},{"line_number":22,"context_line":"from watcher.decision_engine.strategy.selection import default as d_selector"},{"line_number":23,"context_line":"from watcher.decision_engine.strategy import strategies"},{"line_number":24,"context_line":"from watcher.metrics_engine.cluster_model_collector import manager"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_636c2f09","line":21,"range":{"start_line":20,"start_character":54,"end_line":21,"end_character":26},"updated":"2016-06-09 16:18:09.000000000","message":"nit: can you fix this import please?","commit_id":"d8f6da6ec758a388ebfd40584bbe32987578c20a"},{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"0a27cf8282a0a4aa1e767755fcd531bd8fac68c7","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from watcher.common import utils"},{"line_number":19,"context_line":"from watcher.decision_engine.solution import default"},{"line_number":20,"context_line":"from watcher.decision_engine.strategy.context.default import \\"},{"line_number":21,"context_line":"    DefaultStrategyContext"},{"line_number":22,"context_line":"from watcher.decision_engine.strategy.selection import default as d_selector"},{"line_number":23,"context_line":"from watcher.decision_engine.strategy import strategies"},{"line_number":24,"context_line":"from watcher.metrics_engine.cluster_model_collector import manager"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_4020e5f7","line":21,"range":{"start_line":20,"start_character":54,"end_line":21,"end_character":26},"in_reply_to":"7aa08908_636c2f09","updated":"2016-06-10 07:18:54.000000000","message":"Done","commit_id":"d8f6da6ec758a388ebfd40584bbe32987578c20a"}]}