)]}'
{"ceilometer/storage/base.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"6c0175617f5cc92eed39497863fe6f1d5bd6f2ea","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    return sort_keys"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"def convert_bulk_recording(fn):"},{"line_number":75,"context_line":"    def wrapper(conn, data):"},{"line_number":76,"context_line":"        if isinstance(data, list):"},{"line_number":77,"context_line":"            for d in data:"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_26d2fe00","line":74,"updated":"2016-05-09 08:41:19.000000000","message":"You control the whole code base calling record_metering_data, so decorating each function being called is really not a good idea.\n\n\nYour patch overall change the interface record_metering_data, so if any external plugins has that, it\u0027s gonna break, decorator or not.\n\nI\u0027d suggest adding a new method e.g. \"record_metering_bulk\" that is being used if available – falling back to record_metering_data if not.","commit_id":"3a5152b2e3e72a00c1eac6a6589d690c51750910"}]}