)]}'
{"nova/tests/test_driver.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"57a8e54863d17ac8daadebbd6aa6fa220a566fd1","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"        self.assertEquals("},{"line_number":48,"context_line":"            sorted([\u0027key1\u0027, \u0027key2\u0027]),"},{"line_number":49,"context_line":"            sorted(drvs.keys())"},{"line_number":50,"context_line":"        )"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        self.assertDriverInstance("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FdZ8%3D","line":49,"updated":"2013-01-21 19:54:47.000000000","message":"tip: instead of sorted(), I use set()","commit_id":"abaef3b618caeb6c8e55650a96afbf3945ccccfa"}],"nova/virt/driver.py":[{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"89fd692c67109d1a84bd769e3b8ec998dd2093d1","unresolved":false,"context_lines":[{"line_number":42,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"def to_driver_registry(named_driver_config, *args, **kwargs):"},{"line_number":46,"context_line":"    driver_registry \u003d dict()"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    for driver_str in named_driver_config:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Fn2o%3D","line":45,"updated":"2013-01-12 08:52:27.000000000","message":"I\u0027m a bit confused about the name of this method. Its not very self explanatory to me. Perhaps that could be fixed with a different name or a doc comment here?","commit_id":"3ff13fb357f52556b037e488b43749d80c88f071"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"f9670dd8bc51057b13214020aab5d10401f20ad1","unresolved":false,"context_lines":[{"line_number":42,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"def to_driver_registry(named_driver_config, *args, **kwargs):"},{"line_number":46,"context_line":"    driver_registry \u003d dict()"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    for driver_str in named_driver_config:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Fi%2F0%3D","line":45,"updated":"2013-01-16 15:44:36.000000000","message":"Would construct_driver_objects_from_config work better?","commit_id":"3ff13fb357f52556b037e488b43749d80c88f071"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"2fde8a5c7cb37343b17f73f2471ec41949e60712","unresolved":false,"context_lines":[{"line_number":42,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"def to_driver_registry(named_driver_config, *args, **kwargs):"},{"line_number":46,"context_line":"    driver_registry \u003d dict()"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    for driver_str in named_driver_config:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FmzM%3D","line":45,"in_reply_to":"AAAAM3%2F%2Fn2o%3D","updated":"2013-01-14 09:01:59.000000000","message":"Any suggestions?\nFor registry, this was the starting point:\nhttp://martinfowler.com/eaaCatalog/registry.html","commit_id":"3ff13fb357f52556b037e488b43749d80c88f071"}]}