)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14819,"name":"Timofey Durakov","email":"timofei.nd@gmail.com","username":"tdurakov"},"change_message_id":"4742fdb291239c9b007ce154a9f2a9ed42ef511b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Commit b9bae02af2168ad64d3b3d28c97c3853cee73272 introduced"},{"line_number":10,"context_line":"\u0027last_seen_up\u0027 to check service in Liberty. \u0027last_seen_up\u0027"},{"line_number":11,"context_line":"may be null at the beginning. so we still need \u0027created_at\u0027"},{"line_number":12,"context_line":"in this case instead of removing \u0027created_at\u0027."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5a273c294341027fba66a54a540182fc890522db"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"bacf61ea_62a60caf","line":12,"range":{"start_line":11,"start_character":30,"end_line":12,"end_character":46},"updated":"2016-08-02 08:34:08.000000000","message":"could you please rephrase commit message. It\u0027s not very clear what exactly is changed.","commit_id":"03e0d8d95e70cd55ce266e727ed7df831678ea53"},{"author":{"_account_id":9796,"name":"ChangBo Guo","email":"glongwave@gmail.com","username":"gcb"},"change_message_id":"ca8ae47ec4e336c5649f0c63c055c272c3cb85d1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Commit b9bae02af2168ad64d3b3d28c97c3853cee73272 introduced"},{"line_number":10,"context_line":"\u0027last_seen_up\u0027 to check service in Liberty. \u0027last_seen_up\u0027"},{"line_number":11,"context_line":"may be null at the beginning. so we still need \u0027created_at\u0027"},{"line_number":12,"context_line":"in this case instead of removing \u0027created_at\u0027."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5a273c294341027fba66a54a540182fc890522db"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"bacf61ea_684eed3b","line":12,"range":{"start_line":11,"start_character":30,"end_line":12,"end_character":46},"in_reply_to":"bacf61ea_62a60caf","updated":"2016-08-02 09:03:53.000000000","message":"sure. will update in next patchset.","commit_id":"03e0d8d95e70cd55ce266e727ed7df831678ea53"}],"nova/servicegroup/drivers/db.py":[{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"fc138d26cf97a24dbca1050dd6946018fc126f6d","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        \"\"\"Moved from nova.utils"},{"line_number":58,"context_line":"        Check whether a service is up based on last heartbeat."},{"line_number":59,"context_line":"        \"\"\""},{"line_number":60,"context_line":"        last_heartbeat \u003d (service_ref.get(\u0027last_seen_up\u0027) or"},{"line_number":61,"context_line":"            service_ref[\u0027created_at\u0027])"},{"line_number":62,"context_line":"        if isinstance(last_heartbeat, six.string_types):"},{"line_number":63,"context_line":"            # NOTE(russellb) If this service_ref came in over rpc via"}],"source_content_type":"text/x-python","patch_set":7,"id":"bacf61ea_fdfd6109","line":60,"updated":"2016-08-02 15:54:52.000000000","message":"The original change should have probably set a better server default for this one. Alternatively, we could use a hybrid attribute (http://docs.sqlalchemy.org/en/latest/orm/extensions/hybrid.html) to encapsulate logic in the model. But as long as it\u0027s the only place we do this fall back to created_at it\u0027s probably not worth it to do any changes now.","commit_id":"5b718acdad117f7f1ee7a6ff681b1878edcecbb4"},{"author":{"_account_id":9796,"name":"ChangBo Guo","email":"glongwave@gmail.com","username":"gcb"},"change_message_id":"08c76e350e73355ee84853af3ed9d3d215b351c1","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        \"\"\"Moved from nova.utils"},{"line_number":58,"context_line":"        Check whether a service is up based on last heartbeat."},{"line_number":59,"context_line":"        \"\"\""},{"line_number":60,"context_line":"        last_heartbeat \u003d (service_ref.get(\u0027last_seen_up\u0027) or"},{"line_number":61,"context_line":"            service_ref[\u0027created_at\u0027])"},{"line_number":62,"context_line":"        if isinstance(last_heartbeat, six.string_types):"},{"line_number":63,"context_line":"            # NOTE(russellb) If this service_ref came in over rpc via"}],"source_content_type":"text/x-python","patch_set":7,"id":"bacf61ea_ab4a6e25","line":60,"in_reply_to":"bacf61ea_fdfd6109","updated":"2016-08-03 02:59:35.000000000","message":"Thanks , good to know this","commit_id":"5b718acdad117f7f1ee7a6ff681b1878edcecbb4"}]}
