)]}'
{"cinder/db/api.py":[{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"0e1e35521482b6085284ef62c6e45a3fadb0f608","unresolved":false,"context_lines":[{"line_number":1032,"context_line":"def cg_has_volumes_filter(attached_or_with_snapshots\u003dFalse):"},{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":1036,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":1037,"context_line":"    or that have snapshots will be considered."},{"line_number":1038,"context_line":""},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_b1a0d344","line":1037,"range":{"start_line":1035,"start_character":3,"end_line":1037,"end_character":46},"updated":"2016-06-07 09:38:19.000000000","message":"could be repharsed as:\n\n If attached_or_with_snapshots is True, only those volumes are considered which are either in an attached status\n    or which have snapshots.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"6f333dcf39f655873728f05885b8595dc23c2f48","unresolved":false,"context_lines":[{"line_number":1032,"context_line":"def cg_has_volumes_filter(attached_or_with_snapshots\u003dFalse):"},{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":1036,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":1037,"context_line":"    or that have snapshots will be considered."},{"line_number":1038,"context_line":""},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_b613a165","line":1037,"range":{"start_line":1035,"start_character":3,"end_line":1037,"end_character":46},"in_reply_to":"7aa08908_aa6e98fe","updated":"2016-06-07 11:31:50.000000000","message":"oops.. sorry if this is the case. :)\n\nI don\u0027t know but I checked it on multiple online grammar tools to confirm, they are showing it perfectly fine..\n\n1. http://www.gingersoftware.com/grammarcheck#.V1aruPl97IU\n2. http://www.reverso.net/spell-checker/english-spelling-grammar/\n\n\"If attached_or_with_snapshots is True, only those volumes are considered which are either in an attached status or which have snapshots\"","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"af60f1d549dd2641ff9217b7985b805b42d0deab","unresolved":false,"context_lines":[{"line_number":1032,"context_line":"def cg_has_volumes_filter(attached_or_with_snapshots\u003dFalse):"},{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":1036,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":1037,"context_line":"    or that have snapshots will be considered."},{"line_number":1038,"context_line":""},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_aa6e98fe","line":1037,"range":{"start_line":1035,"start_character":3,"end_line":1037,"end_character":46},"in_reply_to":"7aa08908_b1a0d344","updated":"2016-06-07 10:35:24.000000000","message":"Your sentence is grammatically incorrect, but the general phrasing is better, so I\u0027ll rephrase the paragraph.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"59de3b9a83872ceac6946b409b258a95041328d1","unresolved":false,"context_lines":[{"line_number":1035,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":1036,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":1037,"context_line":"    or that have snapshots will be considered."},{"line_number":1038,"context_line":""},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1041,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_91395744","line":1038,"updated":"2016-06-07 09:37:55.000000000","message":"Isn\u0027t this line redundant?","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"0e1e35521482b6085284ef62c6e45a3fadb0f608","unresolved":false,"context_lines":[{"line_number":1035,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":1036,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":1037,"context_line":"    or that have snapshots will be considered."},{"line_number":1038,"context_line":""},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1041,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_71bb0be6","line":1038,"updated":"2016-06-07 09:38:19.000000000","message":"this blank line is not required.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"af60f1d549dd2641ff9217b7985b805b42d0deab","unresolved":false,"context_lines":[{"line_number":1035,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":1036,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":1037,"context_line":"    or that have snapshots will be considered."},{"line_number":1038,"context_line":""},{"line_number":1039,"context_line":"    \"\"\""},{"line_number":1040,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1041,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_0a9c0c68","line":1038,"in_reply_to":"7aa08908_91395744","updated":"2016-06-07 10:35:24.000000000","message":"Yep, it slipped in there.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"59de3b9a83872ceac6946b409b258a95041328d1","unresolved":false,"context_lines":[{"line_number":1048,"context_line":"    create another CG."},{"line_number":1049,"context_line":""},{"line_number":1050,"context_line":"    This filter will not include CGs that have used the ID but have already"},{"line_number":1051,"context_line":"    finished its creation, as in their status is no longer creting."},{"line_number":1052,"context_line":""},{"line_number":1053,"context_line":"    Filter uses a subquery that allows it to be used on updates to the"},{"line_number":1054,"context_line":"    consistencygroups table."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_11c1e72e","line":1051,"range":{"start_line":1051,"start_character":30,"end_line":1051,"end_character":32},"updated":"2016-06-07 09:37:55.000000000","message":"This sentence sounds a little strange with that \"in\".","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"af60f1d549dd2641ff9217b7985b805b42d0deab","unresolved":false,"context_lines":[{"line_number":1048,"context_line":"    create another CG."},{"line_number":1049,"context_line":""},{"line_number":1050,"context_line":"    This filter will not include CGs that have used the ID but have already"},{"line_number":1051,"context_line":"    finished its creation, as in their status is no longer creting."},{"line_number":1052,"context_line":""},{"line_number":1053,"context_line":"    Filter uses a subquery that allows it to be used on updates to the"},{"line_number":1054,"context_line":"    consistencygroups table."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_c5831503","line":1051,"range":{"start_line":1051,"start_character":13,"end_line":1051,"end_character":16},"updated":"2016-06-07 10:35:24.000000000","message":"nit: \"its\" is the singular form and plural should be used as we are referring to a plural noun (CGs), so \"their\" should be used.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"59de3b9a83872ceac6946b409b258a95041328d1","unresolved":false,"context_lines":[{"line_number":1048,"context_line":"    create another CG."},{"line_number":1049,"context_line":""},{"line_number":1050,"context_line":"    This filter will not include CGs that have used the ID but have already"},{"line_number":1051,"context_line":"    finished its creation, as in their status is no longer creting."},{"line_number":1052,"context_line":""},{"line_number":1053,"context_line":"    Filter uses a subquery that allows it to be used on updates to the"},{"line_number":1054,"context_line":"    consistencygroups table."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_71b78bd8","line":1051,"range":{"start_line":1051,"start_character":59,"end_line":1051,"end_character":66},"updated":"2016-06-07 09:37:55.000000000","message":"s/creting/creating","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"af60f1d549dd2641ff9217b7985b805b42d0deab","unresolved":false,"context_lines":[{"line_number":1048,"context_line":"    create another CG."},{"line_number":1049,"context_line":""},{"line_number":1050,"context_line":"    This filter will not include CGs that have used the ID but have already"},{"line_number":1051,"context_line":"    finished its creation, as in their status is no longer creting."},{"line_number":1052,"context_line":""},{"line_number":1053,"context_line":"    Filter uses a subquery that allows it to be used on updates to the"},{"line_number":1054,"context_line":"    consistencygroups table."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_a5808947","line":1051,"range":{"start_line":1051,"start_character":30,"end_line":1051,"end_character":32},"in_reply_to":"7aa08908_11c1e72e","updated":"2016-06-07 10:35:24.000000000","message":"\"As in\" has multiple meanings, it\u0027s mainly used to give more information or clarify, either giving an example or just additional information.  Here it\u0027s being used as an equivalent to \"by which I mean\".\n\nBut I\u0027ll change it, since this may sound weird to more of us non-native speakers.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"af60f1d549dd2641ff9217b7985b805b42d0deab","unresolved":false,"context_lines":[{"line_number":1048,"context_line":"    create another CG."},{"line_number":1049,"context_line":""},{"line_number":1050,"context_line":"    This filter will not include CGs that have used the ID but have already"},{"line_number":1051,"context_line":"    finished its creation, as in their status is no longer creting."},{"line_number":1052,"context_line":""},{"line_number":1053,"context_line":"    Filter uses a subquery that allows it to be used on updates to the"},{"line_number":1054,"context_line":"    consistencygroups table."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_653b419d","line":1051,"range":{"start_line":1051,"start_character":59,"end_line":1051,"end_character":66},"in_reply_to":"7aa08908_71b78bd8","updated":"2016-06-07 10:35:24.000000000","message":"Done","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"503e46bad3e9374aa9bb71cba5ed6948a3cc4204","unresolved":false,"context_lines":[{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    When attached_or_with_snapshots parameter is given a True value only"},{"line_number":1036,"context_line":"    attached volumes and those with snapshots will be considered."},{"line_number":1037,"context_line":"    \"\"\""},{"line_number":1038,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1039,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_5b7d48e6","line":1036,"updated":"2016-06-07 11:09:15.000000000","message":"Shouldn\u0027t this be:\nand -\u003e or\n?","commit_id":"29ecb42452e376b6225e8a16ef44d02cdf1845b4"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"076d9f64e0aedfb8e10f9ed36140944a7d2754fe","unresolved":false,"context_lines":[{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    When attached_or_with_snapshots parameter is given a True value only"},{"line_number":1036,"context_line":"    attached volumes and those with snapshots will be considered."},{"line_number":1037,"context_line":"    \"\"\""},{"line_number":1038,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1039,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_c52ee741","line":1036,"in_reply_to":"7aa08908_3046404f","updated":"2016-06-09 07:58:23.000000000","message":"I don\u0027t consider this nitpicking, if the docstring can be misunderstood in this way it needs to be changed.  Moreover if the main purpose of the patch is to add docstrings.  ;-)\n\nI just wanted to point out why I thought it was grammatically correct, which is irrelevant if it can be misinterpreted.  And that\u0027s why I changed it.","commit_id":"29ecb42452e376b6225e8a16ef44d02cdf1845b4"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"5144daa7fb1d23b33f1ff0f6617f0aedff5822f8","unresolved":false,"context_lines":[{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    When attached_or_with_snapshots parameter is given a True value only"},{"line_number":1036,"context_line":"    attached volumes and those with snapshots will be considered."},{"line_number":1037,"context_line":"    \"\"\""},{"line_number":1038,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1039,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_78ce47b5","line":1036,"in_reply_to":"7aa08908_5b7d48e6","updated":"2016-06-07 11:35:46.000000000","message":"+1","commit_id":"29ecb42452e376b6225e8a16ef44d02cdf1845b4"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"6aa88f4233357daf74a87c56356b8e1ebf162af1","unresolved":false,"context_lines":[{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    When attached_or_with_snapshots parameter is given a True value only"},{"line_number":1036,"context_line":"    attached volumes and those with snapshots will be considered."},{"line_number":1037,"context_line":"    \"\"\""},{"line_number":1038,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1039,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_9d9da868","line":1036,"in_reply_to":"7aa08908_5b7d48e6","updated":"2016-06-08 08:23:25.000000000","message":"Afaik this is correct, it\u0027s like saying \"only Peter and Martin will be considered for the promotion\".  This is a sentence and not an algebraic boolean expression, so AND does not have to follow Boole\u0027s rules.  The parameter name uses OR because that is part of the code and should follow it.  But I\u0027ll change it to avoid possible misunderstandings.","commit_id":"29ecb42452e376b6225e8a16ef44d02cdf1845b4"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"61bc5e31fff93dbf821cfbe40b208cc44c50af62","unresolved":false,"context_lines":[{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    When attached_or_with_snapshots parameter is given a True value only"},{"line_number":1036,"context_line":"    attached volumes and those with snapshots will be considered."},{"line_number":1037,"context_line":"    \"\"\""},{"line_number":1038,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1039,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_369daebc","line":1036,"in_reply_to":"7aa08908_9d9da868","updated":"2016-06-08 08:30:00.000000000","message":"Haven\u0027t we gone too far here…?\n\n:D","commit_id":"29ecb42452e376b6225e8a16ef44d02cdf1845b4"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"d27d2811ed124394f6ba647e4b0b1aff0fd3cf7d","unresolved":false,"context_lines":[{"line_number":1033,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":1034,"context_line":""},{"line_number":1035,"context_line":"    When attached_or_with_snapshots parameter is given a True value only"},{"line_number":1036,"context_line":"    attached volumes and those with snapshots will be considered."},{"line_number":1037,"context_line":"    \"\"\""},{"line_number":1038,"context_line":"    return IMPL.cg_has_volumes_filter(attached_or_with_snapshots)"},{"line_number":1039,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_3046404f","line":1036,"in_reply_to":"7aa08908_9d9da868","updated":"2016-06-08 12:58:39.000000000","message":"OK, do as you like. I normally don\u0027t nit about these things, but it is a comment, so it\u0027s nice if the meaning is clear. But I\u0027ll shut up now :)","commit_id":"29ecb42452e376b6225e8a16ef44d02cdf1845b4"}],"cinder/db/sqlalchemy/api.py":[{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"0e1e35521482b6085284ef62c6e45a3fadb0f608","unresolved":false,"context_lines":[{"line_number":4224,"context_line":"def cg_has_volumes_filter(attached_or_with_snapshots\u003dFalse):"},{"line_number":4225,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":4226,"context_line":""},{"line_number":4227,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":4228,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":4229,"context_line":"    or that have snapshots will be considered."},{"line_number":4230,"context_line":""},{"line_number":4231,"context_line":"    \"\"\""},{"line_number":4232,"context_line":"    query \u003d sql.exists().where("}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_d13bdf36","line":4229,"range":{"start_line":4227,"start_character":0,"end_line":4229,"end_character":46},"updated":"2016-06-07 09:38:19.000000000","message":"can be rephrased like\n\n    If attached_or_with_snapshots is True, then only those volumes are considered which are either in an attached status\n    or which have snapshots.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"0e1e35521482b6085284ef62c6e45a3fadb0f608","unresolved":false,"context_lines":[{"line_number":4227,"context_line":"    If attached_or_with_snapshots is True we will be more specific in which"},{"line_number":4228,"context_line":"    volumes are to be considered and only those that are in an attached status"},{"line_number":4229,"context_line":"    or that have snapshots will be considered."},{"line_number":4230,"context_line":""},{"line_number":4231,"context_line":"    \"\"\""},{"line_number":4232,"context_line":"    query \u003d sql.exists().where("},{"line_number":4233,"context_line":"        and_(models.Volume.consistencygroup_id \u003d\u003d models.ConsistencyGroup.id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_51936f4f","line":4230,"updated":"2016-06-07 09:38:19.000000000","message":"this blank line is not required.","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"0e1e35521482b6085284ef62c6e45a3fadb0f608","unresolved":false,"context_lines":[{"line_number":4250,"context_line":"    create another CG."},{"line_number":4251,"context_line":""},{"line_number":4252,"context_line":"    This filter will not include CGs that have used the ID but have already"},{"line_number":4253,"context_line":"    finished its creation, as in their status is no longer creting."},{"line_number":4254,"context_line":""},{"line_number":4255,"context_line":"    Filter uses a subquery that allows it to be used on updates to the"},{"line_number":4256,"context_line":"    consistencygroups table."}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_f16edb2d","line":4253,"range":{"start_line":4253,"start_character":59,"end_line":4253,"end_character":66},"updated":"2016-06-07 09:38:19.000000000","message":"s/creting/creating","commit_id":"26e4d8d1da304ac36eeee9226365409ff05ba883"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"503e46bad3e9374aa9bb71cba5ed6948a3cc4204","unresolved":false,"context_lines":[{"line_number":4225,"context_line":"    \"\"\"Return a filter to check if a CG has volumes."},{"line_number":4226,"context_line":""},{"line_number":4227,"context_line":"    When attached_or_with_snapshots parameter is given a True value only"},{"line_number":4228,"context_line":"    attached volumes and those with snapshots will be considered."},{"line_number":4229,"context_line":"    \"\"\""},{"line_number":4230,"context_line":"    query \u003d sql.exists().where("},{"line_number":4231,"context_line":"        and_(models.Volume.consistencygroup_id \u003d\u003d models.ConsistencyGroup.id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_3b503c5d","line":4228,"updated":"2016-06-07 11:09:15.000000000","message":"shouldn\u0027t it be:\nand -\u003e or\n?","commit_id":"29ecb42452e376b6225e8a16ef44d02cdf1845b4"}]}