)]}'
{"shade/__init__.py":[{"author":{"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"},"change_message_id":"454ca8ad39b0d520ed94a59455fbc79a2082bcd8","unresolved":false,"context_lines":[{"line_number":69,"context_line":"    return ["},{"line_number":70,"context_line":"        OpenStackCloud("},{"line_number":71,"context_line":"            name\u003df.name, debug\u003ddebug,"},{"line_number":72,"context_line":"            cache_interval\u003dcloud_config.get_cache_max_age(),"},{"line_number":73,"context_line":"            cache_class\u003dcloud_config.get_cache_class(),"},{"line_number":74,"context_line":"            cache_arguments\u003dcloud_config.get_cache_arguments(),"},{"line_number":75,"context_line":"            **f.config)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_7d9512e3","line":72,"updated":"2015-03-11 21:34:43.000000000","message":"s/cloud_config/config/","commit_id":"c413dbb9c8cbe608f1cfd1ed6cd2d1dca1f18818"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"fc27ecfdc3982b48b6132904106b6c78a8d6ceac","unresolved":false,"context_lines":[{"line_number":69,"context_line":"    return ["},{"line_number":70,"context_line":"        OpenStackCloud("},{"line_number":71,"context_line":"            name\u003df.name, debug\u003ddebug,"},{"line_number":72,"context_line":"            cache_interval\u003dcloud_config.get_cache_max_age(),"},{"line_number":73,"context_line":"            cache_class\u003dcloud_config.get_cache_class(),"},{"line_number":74,"context_line":"            cache_arguments\u003dcloud_config.get_cache_arguments(),"},{"line_number":75,"context_line":"            **f.config)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_12d4a91e","line":72,"in_reply_to":"9a80dd14_7d9512e3","updated":"2015-03-13 13:37:04.000000000","message":"Done","commit_id":"c413dbb9c8cbe608f1cfd1ed6cd2d1dca1f18818"},{"author":{"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"},"change_message_id":"5469ad1b86ffc4231383b635eddde28b14418361","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                 private\u003dFalse,"},{"line_number":194,"context_line":"                 verify\u003dTrue, cacert\u003dNone, cert\u003dNone, key\u003dNone,"},{"line_number":195,"context_line":"                 debug\u003dFalse, cache_interval\u003dNone,"},{"line_number":196,"context_line":"                 cache_class\u003d\u0027dogpile.cache.memory\u0027,"},{"line_number":197,"context_line":"                 cache_arguments\u003dNone,"},{"line_number":198,"context_line":"                 **kwargs):"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_a29f8fb3","line":196,"updated":"2015-03-11 09:17:38.000000000","message":"Not really fan of having two conflicting defaults. Could lead to misinterpretations.","commit_id":"c413dbb9c8cbe608f1cfd1ed6cd2d1dca1f18818"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"9be7ea9b72d0563959f57e1a633cbd8316041423","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                 private\u003dFalse,"},{"line_number":194,"context_line":"                 verify\u003dTrue, cacert\u003dNone, cert\u003dNone, key\u003dNone,"},{"line_number":195,"context_line":"                 debug\u003dFalse, cache_interval\u003dNone,"},{"line_number":196,"context_line":"                 cache_class\u003d\u0027dogpile.cache.memory\u0027,"},{"line_number":197,"context_line":"                 cache_arguments\u003dNone,"},{"line_number":198,"context_line":"                 **kwargs):"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_b9bcc2dd","line":196,"updated":"2015-03-06 14:11:27.000000000","message":"Should we make this default to dogpile.cache.null so that a user has to opt-in? If os-client-config is used, it will automatically become opt-in, which may be a confusing, inconsistent behavior change.","commit_id":"c413dbb9c8cbe608f1cfd1ed6cd2d1dca1f18818"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"fc27ecfdc3982b48b6132904106b6c78a8d6ceac","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                 private\u003dFalse,"},{"line_number":194,"context_line":"                 verify\u003dTrue, cacert\u003dNone, cert\u003dNone, key\u003dNone,"},{"line_number":195,"context_line":"                 debug\u003dFalse, cache_interval\u003dNone,"},{"line_number":196,"context_line":"                 cache_class\u003d\u0027dogpile.cache.memory\u0027,"},{"line_number":197,"context_line":"                 cache_arguments\u003dNone,"},{"line_number":198,"context_line":"                 **kwargs):"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_b21d9d11","line":196,"in_reply_to":"9a80dd14_a29f8fb3","updated":"2015-03-13 13:37:04.000000000","message":"Done","commit_id":"c413dbb9c8cbe608f1cfd1ed6cd2d1dca1f18818"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"fc27ecfdc3982b48b6132904106b6c78a8d6ceac","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                 private\u003dFalse,"},{"line_number":194,"context_line":"                 verify\u003dTrue, cacert\u003dNone, cert\u003dNone, key\u003dNone,"},{"line_number":195,"context_line":"                 debug\u003dFalse, cache_interval\u003dNone,"},{"line_number":196,"context_line":"                 cache_class\u003d\u0027dogpile.cache.memory\u0027,"},{"line_number":197,"context_line":"                 cache_arguments\u003dNone,"},{"line_number":198,"context_line":"                 **kwargs):"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_f21395fe","line":196,"in_reply_to":"9a80dd14_b9bcc2dd","updated":"2015-03-13 13:37:04.000000000","message":"Done","commit_id":"c413dbb9c8cbe608f1cfd1ed6cd2d1dca1f18818"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"9688aec44d845df7e76223b9878a3fbe3ce348dc","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                 private\u003dFalse,"},{"line_number":194,"context_line":"                 verify\u003dTrue, cacert\u003dNone, cert\u003dNone, key\u003dNone,"},{"line_number":195,"context_line":"                 debug\u003dFalse, cache_interval\u003dNone,"},{"line_number":196,"context_line":"                 cache_class\u003d\u0027dogpile.cache.memory\u0027,"},{"line_number":197,"context_line":"                 cache_arguments\u003dNone,"},{"line_number":198,"context_line":"                 **kwargs):"},{"line_number":199,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_3960d200","line":196,"in_reply_to":"9a80dd14_b9bcc2dd","updated":"2015-03-06 14:17:45.000000000","message":"Ok, nm. I see that the default will actually become the null backend when cache_interval is None. Sort of a confusing coding path to set the default, but it works and can be altered later.","commit_id":"c413dbb9c8cbe608f1cfd1ed6cd2d1dca1f18818"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"c5b8a97093ca985e1648a1ddf43babf84937c16b","unresolved":false,"context_lines":[{"line_number":180,"context_line":"                               means do not cache at all."},{"line_number":181,"context_line":"                               (optional, defaults to None)"},{"line_number":182,"context_line":"    :param string cache_class: What dogpile.cache cache class to use."},{"line_number":183,"context_line":"                               (optional, defaults to dogpile.cache.memory,"},{"line_number":184,"context_line":"                               unless cache_interval is None)"},{"line_number":185,"context_line":"    :param dict cache_arguments: Additional arguments to pass to the cache"},{"line_number":186,"context_line":"                                 constructor (optional, defaults to None)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_9f36d501","line":183,"updated":"2015-03-19 19:43:21.000000000","message":"Documented default is wrong.","commit_id":"49ce13d4f66901923928cbd33f315e031053191a"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"c5b8a97093ca985e1648a1ddf43babf84937c16b","unresolved":false,"context_lines":[{"line_number":216,"context_line":"        # None a user will not get any in-memory cache and thus not have"},{"line_number":217,"context_line":"        # a large memory leak."},{"line_number":218,"context_line":"        if cache_interval is None:"},{"line_number":219,"context_line":"            cache_class \u003d \u0027dogpile.cache.null\u0027"},{"line_number":220,"context_line":"        self._cache \u003d cache.make_region().configure("},{"line_number":221,"context_line":"            cache_class, expiration_time\u003dcache_interval,"},{"line_number":222,"context_line":"            arguments\u003dcache_arguments)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_9f1df575","line":219,"updated":"2015-03-19 19:43:21.000000000","message":"Seems like this entire \u0027if\u0027 and the comment above can be removed now, yes?","commit_id":"49ce13d4f66901923928cbd33f315e031053191a"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"716c39672fc460879837d353c0a00c6fe5db6c47","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                               means do not cache at all."},{"line_number":188,"context_line":"                               (optional, defaults to None)"},{"line_number":189,"context_line":"    :param string cache_class: What dogpile.cache cache class to use."},{"line_number":190,"context_line":"                               (optional, defaults to dogpile.cache.null)"},{"line_number":191,"context_line":"    :param dict cache_arguments: Additional arguments to pass to the cache"},{"line_number":192,"context_line":"                                 constructor (optional, defaults to None)"},{"line_number":193,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":8,"id":"da9b358b_6ab934e9","line":190,"updated":"2015-03-30 20:34:53.000000000","message":"Super-nit: I know this param says string, but since it is a string which is the name of a class, it may be worth the extra care to quote it in the docs, eg:\n\n   defaults to \"dogpile.cache.null\"","commit_id":"042333c6977e2805adf2a248f6df6ebbb69aeaee"}]}