)]}'
{"mistral/utils/__init__.py":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"b664cfe8d11ebe3f36c23287066c0486a9cdfeba","unresolved":false,"context_lines":[{"line_number":162,"context_line":"    if right is None:"},{"line_number":163,"context_line":"        return left"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"    for key, value in six.iteritems(right):"},{"line_number":166,"context_line":"        left[key] \u003d value"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    return left"},{"line_number":169,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_1a0618ce","line":166,"range":{"start_line":165,"start_character":4,"end_line":166,"end_character":25},"updated":"2016-08-29 09:32:23.000000000","message":"It can be simplified to just left.update(right)","commit_id":"09f01e54d2604cf435f5c36538f27b7a8574ae9e"}],"mistral/workflow/data_flow.py":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"b664cfe8d11ebe3f36c23287066c0486a9cdfeba","unresolved":false,"context_lines":[{"line_number":116,"context_line":"    in_context \u003d (copy.deepcopy(dict(task_ex.in_context))"},{"line_number":117,"context_line":"                  if task_ex.in_context is not None else {})"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    return utils.update_dict(in_context, task_ex.published)"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"def evaluate_workflow_output(wf_spec, ctx):"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_9ab14892","line":119,"range":{"start_line":119,"start_character":17,"end_line":119,"end_character":28},"updated":"2016-08-29 09:32:23.000000000","message":"The alternative is to still use merge_dics() but with overwrite\u003dTrue. But I agree that in this case the behavior would different, instead of replacing the whole variable we would recursively merge it so that we could have:\n\ntask1:\n  publish:\n    my_var: {\u0027k1\u0027: \u0027v1\u0027}\n  on-success: task2\n\ntask2:\n  publish:\n    my_var: {\u0027k2\u0027: \u0027v2\u0027}\n\nand the result would be my_var\u003d{\u0027k1\u0027: \u0027v1\u0027, \u0027k2\u0027: \u0027v2\u0027}\n\nWhich is not what we want.","commit_id":"09f01e54d2604cf435f5c36538f27b7a8574ae9e"}]}
