)]}'
{"trove/taskmanager/models.py":[{"author":{"_account_id":10295,"name":"mariam john","email":"mariamj@us.ibm.com","username":"mariamj"},"change_message_id":"075171cf7f0a8bb0b1980d3cb214e2e0bac30587","unresolved":false,"context_lines":[{"line_number":1554,"context_line":"                                                       needs_server\u003dFalse)"},{"line_number":1555,"context_line":"                    if instance and ("},{"line_number":1556,"context_line":"                            include_clustered or not instance.cluster_id):"},{"line_number":1557,"context_line":"                        reapply_count +\u003d 1"},{"line_number":1558,"context_line":"                        try:"},{"line_number":1559,"context_line":"                            module_models.Modules.validate("},{"line_number":1560,"context_line":"                                modules, instance.datastore.id,"}],"source_content_type":"text/x-python","patch_set":7,"id":"ba5201f7_1362f1ad","line":1557,"updated":"2017-01-09 20:07:23.000000000","message":"Shouldn\u0027t we be incrementing reapply_count inside the try block. What happens if it isn\u0027t able to reapply the module. Similarly, if it gets into the except block, shouldn\u0027t we be incrementing the skipped_count variable? Thanks Peter.","commit_id":"e0d9c0676a9aefcfffaf80fd04151b30e0c2e69f"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"e5502a81114434941a907c233c2087644e828742","unresolved":false,"context_lines":[{"line_number":1554,"context_line":"                                                       needs_server\u003dFalse)"},{"line_number":1555,"context_line":"                    if instance and ("},{"line_number":1556,"context_line":"                            include_clustered or not instance.cluster_id):"},{"line_number":1557,"context_line":"                        reapply_count +\u003d 1"},{"line_number":1558,"context_line":"                        try:"},{"line_number":1559,"context_line":"                            module_models.Modules.validate("},{"line_number":1560,"context_line":"                                modules, instance.datastore.id,"}],"source_content_type":"text/x-python","patch_set":7,"id":"ba5201f7_7243346d","line":1557,"in_reply_to":"ba5201f7_1362f1ad","updated":"2017-01-09 22:16:27.000000000","message":"You\u0027re right. :)  I\u0027ll make the change","commit_id":"e0d9c0676a9aefcfffaf80fd04151b30e0c2e69f"}]}