)]}'
{"monasca/common/repositories/influxdb/metrics_repository.py":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"2ca8c0137b43e72c1ac899eb9e8522b26703fddb","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            LOG.exception()"},{"line_number":48,"context_line":"            raise exceptions.RepositoryException(ex)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def _build_query(self, dimensions, name, tenant_id):"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        query \u003d \u0027list series from /^\u0027"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_abe69be6","line":50,"updated":"2014-10-09 22:44:51.000000000","message":"I think a comment giving an example of a series name would be helpful here","commit_id":"37d490c64262b1801799771fbf9616b35e725282"},{"author":{"_account_id":12512,"name":"Deklan Dieterly","email":"deklan.dieterly@hpe.com","username":"deklan"},"change_message_id":"24082c5ba63a5d92d1e2b813ed416383ff427041","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            LOG.exception()"},{"line_number":48,"context_line":"            raise exceptions.RepositoryException(ex)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    def _build_query(self, dimensions, name, tenant_id):"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        query \u003d \u0027list series from /^\u0027"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_7b570f27","line":50,"in_reply_to":"baa201ad_abe69be6","updated":"2014-10-10 14:38:06.000000000","message":"funny, once again that thought went thru my mind, but i was too lazy to do it.  i\u0027ll add it.","commit_id":"37d490c64262b1801799771fbf9616b35e725282"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"2ca8c0137b43e72c1ac899eb9e8522b26703fddb","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    def _decode_influxdb_serie_names(self, series_names):"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        json_metric_list \u003d []"},{"line_number":91,"context_line":"        serie_names_list_list \u003d series_names[0][\u0027points\u0027]"},{"line_number":92,"context_line":"        for serie_name_list in serie_names_list_list:"},{"line_number":93,"context_line":"            serie_name \u003d serie_name_list[1]"},{"line_number":94,"context_line":"            match \u003d self._serie_name_reqex.match(serie_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_ab813bfd","line":91,"updated":"2014-10-09 22:44:51.000000000","message":"What happens if no series names were matched? Do you need to check for that before trying to access series_names[0]. Maybe a comment giving an example of the returned json would be helpful","commit_id":"37d490c64262b1801799771fbf9616b35e725282"},{"author":{"_account_id":12512,"name":"Deklan Dieterly","email":"deklan.dieterly@hpe.com","username":"deklan"},"change_message_id":"24082c5ba63a5d92d1e2b813ed416383ff427041","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    def _decode_influxdb_serie_names(self, series_names):"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        json_metric_list \u003d []"},{"line_number":91,"context_line":"        serie_names_list_list \u003d series_names[0][\u0027points\u0027]"},{"line_number":92,"context_line":"        for serie_name_list in serie_names_list_list:"},{"line_number":93,"context_line":"            serie_name \u003d serie_name_list[1]"},{"line_number":94,"context_line":"            match \u003d self._serie_name_reqex.match(serie_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_1bbb6373","line":91,"in_reply_to":"baa201ad_ab813bfd","updated":"2014-10-10 14:38:06.000000000","message":"if no series names are matched, then \u0027[]\u0027 is returned.","commit_id":"37d490c64262b1801799771fbf9616b35e725282"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"2ca8c0137b43e72c1ac899eb9e8522b26703fddb","unresolved":false,"context_lines":[{"line_number":102,"context_line":"                    # remove the name, tenant_id, and region; just"},{"line_number":103,"context_line":"                    # dimensions remain"},{"line_number":104,"context_line":"                    dimensions_part \u003d \\"},{"line_number":105,"context_line":"                        self._serie_name_tenant_id_region_regex.sub("},{"line_number":106,"context_line":"                        \u0027\u0027, serie_name)"},{"line_number":107,"context_line":"                    dimensions \u003d {}"},{"line_number":108,"context_line":"                    dimension_list \u003d self._serie_name_dimension_regex.findall("}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_cb103fab","line":105,"updated":"2014-10-09 22:44:51.000000000","message":"Why isn\u0027t dimensions_part the same as dimensions? Seems like you have already pulled this out","commit_id":"37d490c64262b1801799771fbf9616b35e725282"},{"author":{"_account_id":12512,"name":"Deklan Dieterly","email":"deklan.dieterly@hpe.com","username":"deklan"},"change_message_id":"24082c5ba63a5d92d1e2b813ed416383ff427041","unresolved":false,"context_lines":[{"line_number":102,"context_line":"                    # remove the name, tenant_id, and region; just"},{"line_number":103,"context_line":"                    # dimensions remain"},{"line_number":104,"context_line":"                    dimensions_part \u003d \\"},{"line_number":105,"context_line":"                        self._serie_name_tenant_id_region_regex.sub("},{"line_number":106,"context_line":"                        \u0027\u0027, serie_name)"},{"line_number":107,"context_line":"                    dimensions \u003d {}"},{"line_number":108,"context_line":"                    dimension_list \u003d self._serie_name_dimension_regex.findall("}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_bb497790","line":105,"in_reply_to":"baa201ad_cb103fab","updated":"2014-10-10 14:38:06.000000000","message":"it\u0027s not the same.  it\u0027s complicated. this was necessary because of the way python handles multiple group matches.  swing by my desk for more details.","commit_id":"37d490c64262b1801799771fbf9616b35e725282"}]}