)]}'
{"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"change_message_id":"99e58ebef086249f1d62982c2190e75554e12eb7","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        # OK, filtering on this key; what value do we search for?"},{"line_number":241,"context_line":"        value \u003d filters.pop(key)"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"        if key \u003d\u003d \u0027metadata\u0027 or key \u003d\u003d \u0027system_metadata\u0027:"},{"line_number":244,"context_line":"            column_attr \u003d getattr(model, key)"},{"line_number":245,"context_line":"            if isinstance(value, list):"},{"line_number":246,"context_line":"                for item in value:"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATn%2F%2FYY8%3D","line":243,"updated":"2013-11-07 17:22:09.000000000","message":"Why not ?\n\n    key in (\u0027metadata\u0027, \u0027system_metadata\u0027):","commit_id":"1efdd075299c998ae90de5c3bbee2344f93888b2"},{"author":{"_account_id":91,"name":"Alvaro","email":"aloga@ifca.unican.es","username":"aloga"},"change_message_id":"b6ca3e88cbeaec8f64cdddb2d44805e282a25885","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        # OK, filtering on this key; what value do we search for?"},{"line_number":241,"context_line":"        value \u003d filters.pop(key)"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"        if key \u003d\u003d \u0027metadata\u0027 or key \u003d\u003d \u0027system_metadata\u0027:"},{"line_number":244,"context_line":"            column_attr \u003d getattr(model, key)"},{"line_number":245,"context_line":"            if isinstance(value, list):"},{"line_number":246,"context_line":"                for item in value:"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATn%2F%2FPiw%3D","line":243,"in_reply_to":"AAAATn%2F%2FX1Q%3D","updated":"2013-11-12 11:30:53.000000000","message":"Yes, indeed. Sorry for bothering!","commit_id":"1efdd075299c998ae90de5c3bbee2344f93888b2"},{"author":{"_account_id":1313,"name":"Yaguang Tang","email":"heut2008@gmail.com","username":"heut2008"},"change_message_id":"7112df6e2ef6550b0a4b7df08e1b7b2fa073b6ae","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        # OK, filtering on this key; what value do we search for?"},{"line_number":241,"context_line":"        value \u003d filters.pop(key)"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"        if key \u003d\u003d \u0027metadata\u0027 or key \u003d\u003d \u0027system_metadata\u0027:"},{"line_number":244,"context_line":"            column_attr \u003d getattr(model, key)"},{"line_number":245,"context_line":"            if isinstance(value, list):"},{"line_number":246,"context_line":"                for item in value:"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAATn%2F%2FX1Q%3D","line":243,"in_reply_to":"AAAATn%2F%2FYY8%3D","updated":"2013-11-08 01:15:46.000000000","message":"this just a cherry pick from upstream, and we try to avoid changing patches if not a must, your suggestion is absolutely better.","commit_id":"1efdd075299c998ae90de5c3bbee2344f93888b2"}]}