)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"2c4eb90f85688c0f87545ff4304760f10d88997e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"* Linearized control flow"},{"line_number":10,"context_line":"* Removed nested functions"},{"line_number":11,"context_line":"* Removed nested loop in filter algorithm; O(kn) becomes O(k+n)"},{"line_number":12,"context_line":"* Dictionary no longer requires hashing a tuple"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Id1580348d4c2ff23d750363fa9a03576080c88c0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3a50d1a3_150e91e5","line":12,"updated":"2015-07-24 08:33:32.000000000","message":"I think your change is really good but since it modifies a lot of stuff, it\u0027s a bit difficult to review it so I would prefer to have it splitted into multiple patches, one per item you said above.\n\nAlso, I know it\u0027s a pure refactoring change, but I want to make sure we don\u0027t need to modify unittests also.","commit_id":"feb719f35289d9fa8148961f484cdf35ebee97e3"}],"nova/scheduler/host_manager.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a3f853775a8e093ed02f9697d2638b5c3d70ab7c","unresolved":false,"context_lines":[{"line_number":467,"context_line":""},{"line_number":468,"context_line":"        if force_hosts:"},{"line_number":469,"context_line":"            # Force scheduler to select designated hosts"},{"line_number":470,"context_line":"            filtered_hosts \u003d [host for host in filtered_hosts"},{"line_number":471,"context_line":"                    if host.host in force_hosts]"},{"line_number":472,"context_line":""},{"line_number":473,"context_line":"            # Log hosts that were retained in this filter"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba8a016a_e8b5497e","line":470,"updated":"2015-11-16 15:03:36.000000000","message":"As per Brian\u0027s comments on the previous patch, I think we should look at pulling out the for loop around filtered hosts in this patch.\n\nit looks like that should work, although generates very different log messages, and would be harder to debug... hmm.","commit_id":"01b66ef44df057bac90f219acb00f3b648448a8d"}]}