)]}'
{"tooz/drivers/redis.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"d36258d9ae57bc0428a6fbc9e20c93d869dea8db","unresolved":false,"context_lines":[{"line_number":343,"context_line":"        # sentinel arg."},{"line_number":344,"context_line":"        if \u0027sentinel\u0027 in kwargs:"},{"line_number":345,"context_line":"            sentinel_hosts \u003d [tuple(fallback.split(\u0027:\u0027))"},{"line_number":346,"context_line":"                              for fallback in kwargs[\u0027sentinel_fallback\u0027]]"},{"line_number":347,"context_line":"            sentinel_hosts.insert(0, (kwargs[\u0027host\u0027], kwargs[\u0027port\u0027]))"},{"line_number":348,"context_line":"            sentinel_server \u003d sentinel.Sentinel("},{"line_number":349,"context_line":"                sentinel_hosts,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_0b0cf23a","line":346,"updated":"2015-01-12 14:07:36.000000000","message":"Quick question ... is \u0027sentinel_fallback\u0027 an optional query param in the URL, even when the \u0027sentinel\u0027 param is present?\n\ni.e. in the case that it\u0027s absent from the URL, does kwargs[\u0027sentinel_fallback\u0027] evaluate as an empty list, or does it need to be explicitly defaulted as in kwargs.get(\u0027sentinel_fallback\u0027, [])?","commit_id":"4f700ae05d4a30d3b438ddae91c19a8e4d55081a"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"c8a125f2ee2af869e613fede83fe91fa49d8740b","unresolved":false,"context_lines":[{"line_number":343,"context_line":"        # sentinel arg."},{"line_number":344,"context_line":"        if \u0027sentinel\u0027 in kwargs:"},{"line_number":345,"context_line":"            sentinel_hosts \u003d [tuple(fallback.split(\u0027:\u0027))"},{"line_number":346,"context_line":"                              for fallback in kwargs[\u0027sentinel_fallback\u0027]]"},{"line_number":347,"context_line":"            sentinel_hosts.insert(0, (kwargs[\u0027host\u0027], kwargs[\u0027port\u0027]))"},{"line_number":348,"context_line":"            sentinel_server \u003d sentinel.Sentinel("},{"line_number":349,"context_line":"                sentinel_hosts,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_4bcb5a4a","line":346,"in_reply_to":"3a961159_0b0cf23a","updated":"2015-01-12 14:18:29.000000000","message":"Good catch. Fixed in the next patchset, with appropriately updated tests.","commit_id":"4f700ae05d4a30d3b438ddae91c19a8e4d55081a"}]}