)]}'
{"cinder/db/sqlalchemy/api.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"fe5d517bf7db7e031bfab00c24076a9e606cdd4e","unresolved":false,"context_lines":[{"line_number":2594,"context_line":""},{"line_number":2595,"context_line":"    query \u003d model_query(context, models.Transfer).\\"},{"line_number":2596,"context_line":"        filter(models.Volume.id \u003d\u003d models.Transfer.volume_id).\\"},{"line_number":2597,"context_line":"        filter(models.Volume.project_id \u003d\u003d project_id)"},{"line_number":2598,"context_line":"    results \u003d query.all()"},{"line_number":2599,"context_line":"    return _translate_transfers(results)"},{"line_number":2600,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F81S0%3D","line":2597,"updated":"2013-12-19 19:24:59.000000000","message":"If we\u0027re using the more complex filter over filter_by here do we want to go ahead and use the compact statement form here:\nfilter(models.volume.id \u003d\u003d models.Transfer.volume_id, \n       models.volume.project_id \u003d\u003d project_id)\n\nThis isn\u0027t an issue, just a question, something to think about and maybe there\u0027s a good reason not to that you could teach me :)","commit_id":"b5fa6421aba36ae0dbcb6405508cc696acfa068d"}]}