)]}'
{"files/log-gearman-client.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"75e3465a274dae79c7dbe2ad1d2686a9b2cc2de4","unresolved":false,"context_lines":[{"line_number":127,"context_line":"                            key.startswith(self.auto_forward_field_prefix)]"},{"line_number":128,"context_line":"            for forward_key in forward_keys:"},{"line_number":129,"context_line":"                field_key \u003d forward_key.replace(self.auto_forward_field_prefix,"},{"line_number":130,"context_line":"                                                \u0027\u0027, 1)"},{"line_number":131,"context_line":"                fields[field_key] \u003d parameters[forward_key]"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"        return fields"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_0fb059fc","line":130,"updated":"2015-09-10 22:24:52.000000000","message":"I am torn between preserving the prefix and removing it. The code above changes the names so removing the prefix should be fine.","commit_id":"86315785bab2abd4ae81a3973cd46e5d8b0c0b6e"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"75e3465a274dae79c7dbe2ad1d2686a9b2cc2de4","unresolved":false,"context_lines":[{"line_number":173,"context_line":"                self.config[\u0027source-files\u0027], self.source_url)"},{"line_number":174,"context_line":"            self.processors.append(log_processor)"},{"line_number":175,"context_line":"            if \u0027subunit-files\u0027 in self.config:"},{"line_number":176,"context_line":"                field_prefix \u003d self.config.get(\u0027auto-forward-field-prefix\u0027)"},{"line_number":177,"context_line":"                subunit_processor \u003d EventProcessor("},{"line_number":178,"context_line":"                    publisher, gearclient,"},{"line_number":179,"context_line":"                    self.config[\u0027subunit-files\u0027], self.source_url,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_2fac3d9e","line":176,"updated":"2015-09-10 22:24:52.000000000","message":"Field prefix should apply to all Jenkins event processors not just subunit.","commit_id":"86315785bab2abd4ae81a3973cd46e5d8b0c0b6e"},{"author":{"_account_id":6039,"name":"Joni Harker","email":"code@gentlydownthe.net","username":"jesusaurus"},"change_message_id":"2b8b7cf8bb162ccfed410dc97b5ce918badde5f6","unresolved":false,"context_lines":[{"line_number":173,"context_line":"                self.config[\u0027source-files\u0027], self.source_url)"},{"line_number":174,"context_line":"            self.processors.append(log_processor)"},{"line_number":175,"context_line":"            if \u0027subunit-files\u0027 in self.config:"},{"line_number":176,"context_line":"                field_prefix \u003d self.config.get(\u0027auto-forward-field-prefix\u0027)"},{"line_number":177,"context_line":"                subunit_processor \u003d EventProcessor("},{"line_number":178,"context_line":"                    publisher, gearclient,"},{"line_number":179,"context_line":"                    self.config[\u0027subunit-files\u0027], self.source_url,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_0f08597f","line":176,"in_reply_to":"da20952f_2fac3d9e","updated":"2015-09-10 22:33:20.000000000","message":"I agree, this should also be passed in to the log_processor, not just the subunit_processor.","commit_id":"86315785bab2abd4ae81a3973cd46e5d8b0c0b6e"},{"author":{"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},"change_message_id":"757ee419e67b4a0a9ef61d931da346b6f77ba3e6","unresolved":false,"context_lines":[{"line_number":136,"context_line":"            fields[\"node_provider\"] \u003d \"UNKNOWN\""},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"        if self.auto_forward_field_prefix:"},{"line_number":139,"context_line":"            forward_keys \u003d [key for key in parameters.iterkeys() if"},{"line_number":140,"context_line":"                            key.startswith(self.auto_forward_field_prefix)]"},{"line_number":141,"context_line":"            for forward_key in forward_keys:"},{"line_number":142,"context_line":"                fields[forward_key] \u003d parameters[forward_key]"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a57f1b5_3de7ffe5","line":139,"range":{"start_line":139,"start_character":54,"end_line":139,"end_character":62},"updated":"2016-02-10 10:58:01.000000000","message":"For python3 compatibility should use just .keys().","commit_id":"eb0e23ab91d61305e79afb2078120bf8b67425ba"}]}
