)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"aa55c83d67f7732321cc30c364b6ed8694ecb2f7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Lower down the range for columns values  which are being used as uuid."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently column data types are too large for uuids.This patch change the range from 255"},{"line_number":10,"context_line":"characters to 128 characters for columns resource_id, project_id, user_id."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: #1353039"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a016987_7bd61529","line":9,"updated":"2015-05-22 10:42:00.000000000","message":"too long string please make it less or eq 79 chars","commit_id":"8ead23d5ba8f01c97b13ad4a7e8e7fef4210ef5c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"43dbd34847558700b34a313daa5de2e03bc32f4d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently column data types are too large for uuids."},{"line_number":10,"context_line":"This patch change the range from 255 characters to 128"},{"line_number":11,"context_line":"characters for columns resource_id, project_id, user_id."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1353039"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"1af86dd1_0c9576e3","line":11,"updated":"2015-06-12 02:39:50.000000000","message":"I think you should modify commit msg as this is changing other UUID also.","commit_id":"17826d715235fcacd87b5a10fa0750cb0bf9b24b"}],"ceilometer/storage/sqlalchemy/migrate_repo/versions/043_reduce_uuid_data_types.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"0beae2ca6c15608d1e916557bf07fde7e878feb7","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    alarm_history.c.on_behalf_of.alter(type\u003dString(128))"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"def downgrade(migrate_engine):"},{"line_number":41,"context_line":"    meta \u003d MetaData(bind\u003dmigrate_engine)"},{"line_number":42,"context_line":"    resource \u003d Table(\u0027resource\u0027, meta, autoload\u003dTrue)"},{"line_number":43,"context_line":"    resource.c.user_id.alter(type\u003dString(255))"}],"source_content_type":"text/x-python","patch_set":6,"id":"3afb71cf_fb5b0c84","line":40,"range":{"start_line":40,"start_character":14,"end_line":40,"end_character":28},"updated":"2015-06-11 03:41:16.000000000","message":"i think we can remove this downgrade path.\n\nhttps://github.com/openstack/openstack-specs/blob/master/specs/no-downward-sql-migration.rst","commit_id":"49798c1fd4c5dc042951c52e3718513f47dde151"}],"ceilometer/storage/sqlalchemy/models.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"cd4ad0e3bb6105043da479bd9935e4d4b969b8fd","unresolved":false,"context_lines":[{"line_number":285,"context_line":"    )"},{"line_number":286,"context_line":"    event_id \u003d Column(String(128), primary_key\u003dTrue)"},{"line_number":287,"context_line":"    alarm_id \u003d Column(String(128))"},{"line_number":288,"context_line":"    on_behalf_of \u003d Column(String(255))"},{"line_number":289,"context_line":"    project_id \u003d Column(String(128))"},{"line_number":290,"context_line":"    user_id \u003d Column(String(128))"},{"line_number":291,"context_line":"    type \u003d Column(String(20))"}],"source_content_type":"text/x-python","patch_set":5,"id":"3afb71cf_8891d749","line":288,"updated":"2015-06-09 09:04:02.000000000","message":"this is an id","commit_id":"efa5bf67fc1f2aee4c5bf3384beb622f4dd5302c"},{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"426ed8b804aefebd2cca09bf72713b299f0a7bdb","unresolved":false,"context_lines":[{"line_number":285,"context_line":"    )"},{"line_number":286,"context_line":"    event_id \u003d Column(String(128), primary_key\u003dTrue)"},{"line_number":287,"context_line":"    alarm_id \u003d Column(String(128))"},{"line_number":288,"context_line":"    on_behalf_of \u003d Column(String(255))"},{"line_number":289,"context_line":"    project_id \u003d Column(String(128))"},{"line_number":290,"context_line":"    user_id \u003d Column(String(128))"},{"line_number":291,"context_line":"    type \u003d Column(String(20))"}],"source_content_type":"text/x-python","patch_set":5,"id":"3afb71cf_5ba9648f","line":288,"in_reply_to":"3afb71cf_8891d749","updated":"2015-06-09 10:23:43.000000000","message":"you are talking about \u0027on_behalf_of\u0027 right, i am lowering it to 128.","commit_id":"efa5bf67fc1f2aee4c5bf3384beb622f4dd5302c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"43dbd34847558700b34a313daa5de2e03bc32f4d","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    )"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    internal_id \u003d Column(Integer, primary_key\u003dTrue)"},{"line_number":180,"context_line":"    user_id \u003d Column(String(128))"},{"line_number":181,"context_line":"    project_id \u003d Column(String(128))"},{"line_number":182,"context_line":"    source_id \u003d Column(String(128))"},{"line_number":183,"context_line":"    resource_id \u003d Column(String(128), nullable\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1af86dd1_cc9e3e07","line":180,"updated":"2015-06-12 02:39:50.000000000","message":"Why not String(36) as per UUID standard length?\n\nSame for other place.","commit_id":"17826d715235fcacd87b5a10fa0750cb0bf9b24b"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"2b07b97146f8ba22ff0e16e99ea634a6323b7579","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    )"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    internal_id \u003d Column(Integer, primary_key\u003dTrue)"},{"line_number":180,"context_line":"    user_id \u003d Column(String(128))"},{"line_number":181,"context_line":"    project_id \u003d Column(String(128))"},{"line_number":182,"context_line":"    source_id \u003d Column(String(128))"},{"line_number":183,"context_line":"    resource_id \u003d Column(String(128), nullable\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1af86dd1_8048072f","line":180,"in_reply_to":"1af86dd1_cc9e3e07","updated":"2015-06-12 07:56:20.000000000","message":"I also would like to know the reason for 128.\n\nI found keystone generates user_id and project_id in 32 characters (by UUID + hex), but nova and neutron prepared 255 character space in the DB models.\nThose ids will be generated in keystone, so 32 or 36 might be OK.\nIdeally, there should be agreement on size of ids with keystone, otherwise we might faced serious situation once they start to uses longer IDs.","commit_id":"17826d715235fcacd87b5a10fa0750cb0bf9b24b"},{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"cbb31b9614f8f5a3b6dcc9e300fc0a72e23e3ed8","unresolved":false,"context_lines":[{"line_number":217,"context_line":"    timestamp \u003d Column(PreciseTimestamp(), default\u003dlambda: timeutils.utcnow())"},{"line_number":218,"context_line":"    recorded_at \u003d Column(PreciseTimestamp(),"},{"line_number":219,"context_line":"                         default\u003dlambda: timeutils.utcnow())"},{"line_number":220,"context_line":"    message_signature \u003d Column(String(64))"},{"line_number":221,"context_line":"    message_id \u003d Column(String(128))"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"3afb71cf_5c4a0c56","line":220,"updated":"2015-06-11 10:51:16.000000000","message":"Hi why do you decrease message_signature to 64 chars when all other fields were decreased to 128 chars?","commit_id":"17826d715235fcacd87b5a10fa0750cb0bf9b24b"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"2b07b97146f8ba22ff0e16e99ea634a6323b7579","unresolved":false,"context_lines":[{"line_number":217,"context_line":"    timestamp \u003d Column(PreciseTimestamp(), default\u003dlambda: timeutils.utcnow())"},{"line_number":218,"context_line":"    recorded_at \u003d Column(PreciseTimestamp(),"},{"line_number":219,"context_line":"                         default\u003dlambda: timeutils.utcnow())"},{"line_number":220,"context_line":"    message_signature \u003d Column(String(64))"},{"line_number":221,"context_line":"    message_id \u003d Column(String(128))"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"1af86dd1_db082e2b","line":220,"in_reply_to":"3afb71cf_5c4a0c56","updated":"2015-06-12 07:56:20.000000000","message":"message_signature will hold only 64 hex characters which are generated by sha256.\n\nhttps://github.com/openstack/ceilometer/blob/master/ceilometer/publisher/utils.py#L43-58","commit_id":"17826d715235fcacd87b5a10fa0750cb0bf9b24b"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"2b07b97146f8ba22ff0e16e99ea634a6323b7579","unresolved":false,"context_lines":[{"line_number":218,"context_line":"    recorded_at \u003d Column(PreciseTimestamp(),"},{"line_number":219,"context_line":"                         default\u003dlambda: timeutils.utcnow())"},{"line_number":220,"context_line":"    message_signature \u003d Column(String(64))"},{"line_number":221,"context_line":"    message_id \u003d Column(String(128))"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"class FullSample(Base):"}],"source_content_type":"text/x-python","patch_set":7,"id":"1af86dd1_76515162","line":221,"updated":"2015-06-12 07:56:20.000000000","message":"This ID will be created in Ceilometer (sample.py uuid.uuid1()), so we can set minimum length i.e. 36.","commit_id":"17826d715235fcacd87b5a10fa0750cb0bf9b24b"}]}