)]}'
{"searchlight/cmd/manage.py":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"553e7dd0a7b332932961e186abdf1c511aeb30ce","unresolved":false,"context_lines":[{"line_number":283,"context_line":"                def _format_plugin(plugin, indent\u003d0):"},{"line_number":284,"context_line":"                    plugin_doc_type \u003d plugin.get_document_type()"},{"line_number":285,"context_line":"                    handler \u003d plugin.get_notification_handler()"},{"line_number":286,"context_line":"                    event_list \u003d None"},{"line_number":287,"context_line":"                    if handler:"},{"line_number":288,"context_line":"                        event_list \u003d ("},{"line_number":289,"context_line":"                            handler.get_notification_supported_events())"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_02be10ff","line":286,"range":{"start_line":286,"start_character":20,"end_line":286,"end_character":30},"updated":"2016-11-14 15:55:49.000000000","message":"Think event_list and the following three lines could be removed now","commit_id":"01b98328c74d5ae42fd7c224b4d6d68dc15b9214"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"1d5d922d9c252675fba4770ff9d297604c538390","unresolved":false,"context_lines":[{"line_number":283,"context_line":"                def _format_plugin(plugin, indent\u003d0):"},{"line_number":284,"context_line":"                    plugin_doc_type \u003d plugin.get_document_type()"},{"line_number":285,"context_line":"                    handler \u003d plugin.get_notification_handler()"},{"line_number":286,"context_line":"                    event_list \u003d None"},{"line_number":287,"context_line":"                    if handler:"},{"line_number":288,"context_line":"                        event_list \u003d ("},{"line_number":289,"context_line":"                            handler.get_notification_supported_events())"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a77a97e_6726d8cf","line":286,"range":{"start_line":286,"start_character":20,"end_line":286,"end_character":30},"in_reply_to":"9a629dbe_02be10ff","updated":"2016-11-16 01:03:36.000000000","message":"Done","commit_id":"01b98328c74d5ae42fd7c224b4d6d68dc15b9214"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"553e7dd0a7b332932961e186abdf1c511aeb30ce","unresolved":false,"context_lines":[{"line_number":293,"context_line":"                                            plugin.resource_group_name)"},{"line_number":294,"context_line":"                    if plugin_doc_type in es_reindex:"},{"line_number":295,"context_line":"                        display +\u003d \u0027 *\u0027"},{"line_number":296,"context_line":"                    if not event_list:"},{"line_number":297,"context_line":"                        display +\u003d \u0027 !!\u0027"},{"line_number":298,"context_line":"                        plugins_without_notifications.append(plugin)"},{"line_number":299,"context_line":"                    return display + \u0027\u0027.join(_format_plugin(c, indent + 1)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_82406009","line":296,"range":{"start_line":296,"start_character":0,"end_line":296,"end_character":38},"updated":"2016-11-14 15:55:49.000000000","message":"Replace with \u0027if not handler\u0027","commit_id":"01b98328c74d5ae42fd7c224b4d6d68dc15b9214"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"1d5d922d9c252675fba4770ff9d297604c538390","unresolved":false,"context_lines":[{"line_number":293,"context_line":"                                            plugin.resource_group_name)"},{"line_number":294,"context_line":"                    if plugin_doc_type in es_reindex:"},{"line_number":295,"context_line":"                        display +\u003d \u0027 *\u0027"},{"line_number":296,"context_line":"                    if not event_list:"},{"line_number":297,"context_line":"                        display +\u003d \u0027 !!\u0027"},{"line_number":298,"context_line":"                        plugins_without_notifications.append(plugin)"},{"line_number":299,"context_line":"                    return display + \u0027\u0027.join(_format_plugin(c, indent + 1)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a77a97e_872134b7","line":296,"range":{"start_line":296,"start_character":0,"end_line":296,"end_character":38},"in_reply_to":"9a629dbe_82406009","updated":"2016-11-16 01:03:36.000000000","message":"Done","commit_id":"01b98328c74d5ae42fd7c224b4d6d68dc15b9214"}],"searchlight/elasticsearch/plugins/base.py":[{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"553e7dd0a7b332932961e186abdf1c511aeb30ce","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            if self.NotificationHandlerCls:"},{"line_number":188,"context_line":"                version \u003d self.NotificationHandlerCls.get_version(document)"},{"line_number":189,"context_line":"            else:"},{"line_number":190,"context_line":"                version \u003d NotificationBase.get_version(document)"},{"line_number":191,"context_line":"            versions.append(version)"},{"line_number":192,"context_line":"        self.index_helper.save_documents(documents, versions\u003dversions,"},{"line_number":193,"context_line":"                                         index\u003dindex_name)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_a25ba42d","line":190,"range":{"start_line":190,"start_character":26,"end_line":190,"end_character":42},"updated":"2016-11-14 15:55:49.000000000","message":"Can\u0027t remember why get_version is in NotificationBase at all. I think it should get moved out sometime but not as part of this patch","commit_id":"01b98328c74d5ae42fd7c224b4d6d68dc15b9214"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"1d5d922d9c252675fba4770ff9d297604c538390","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            if self.NotificationHandlerCls:"},{"line_number":188,"context_line":"                version \u003d self.NotificationHandlerCls.get_version(document)"},{"line_number":189,"context_line":"            else:"},{"line_number":190,"context_line":"                version \u003d NotificationBase.get_version(document)"},{"line_number":191,"context_line":"            versions.append(version)"},{"line_number":192,"context_line":"        self.index_helper.save_documents(documents, versions\u003dversions,"},{"line_number":193,"context_line":"                                         index\u003dindex_name)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a77a97e_3258a032","line":190,"range":{"start_line":190,"start_character":26,"end_line":190,"end_character":42},"in_reply_to":"9a629dbe_a25ba42d","updated":"2016-11-16 01:03:36.000000000","message":"Yeah, if no subclass overriding, it can be moved out.","commit_id":"01b98328c74d5ae42fd7c224b4d6d68dc15b9214"}]}