)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"428529fe27c944b646b2954c7ef8c50259c18dce","unresolved":false,"context_lines":[{"line_number":13,"context_line":"- Meter"},{"line_number":14,"context_line":"- Project"},{"line_number":15,"context_line":"- Resource"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I8846d7bf6fe2030a05cd5a42ff4bee6bc21bc4e4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAP3%2F%2FzXc%3D","line":16,"updated":"2013-07-11 07:10:58.000000000","message":"missing blueprint line","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"}],"ceilometer/storage/sqlalchemy/models.py":[{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":107,"context_line":"class Meter(Base):"},{"line_number":108,"context_line":"    \"\"\"Metering data.\"\"\""},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    __tablename__ \u003d \u0027meter\u0027"},{"line_number":111,"context_line":"    id \u003d Column(Integer(11), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":112,"context_line":"    counter_name \u003d Column(String(255), nullable\u003dTrue)"},{"line_number":113,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzQI%3D","line":110,"updated":"2013-07-11 07:46:45.000000000","message":"Index \u0027idx_meter_rid_cname\u0027 is missing (10 migration).\nAdd this index in __table_args__.\nAdd all indexes in table for column with index\u003dTrue to __table_args__.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5fe09e3250ae32286a58781477bf0da7112ce94","unresolved":false,"context_lines":[{"line_number":108,"context_line":"    \"\"\"Metering data.\"\"\""},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    __tablename__ \u003d \u0027meter\u0027"},{"line_number":111,"context_line":"    id \u003d Column(Integer(11), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":112,"context_line":"    counter_name \u003d Column(String(255), nullable\u003dTrue)"},{"line_number":113,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"},{"line_number":114,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), nullable\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzW8%3D","line":111,"updated":"2013-07-11 07:25:03.000000000","message":"1) a PK constraint implies nullable\u003dFalse (by definition) and index\u003dTrue (in all RDBMS we use - MySQL, PostgreSQL, SQLite...) - not sure these two are needed here\n\n2) what does 11 means here? According to SQLAlchemy docs this is not an Integer type specific argument","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":111,"context_line":"    id \u003d Column(Integer(11), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":112,"context_line":"    counter_name \u003d Column(String(255), nullable\u003dTrue)"},{"line_number":113,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"},{"line_number":114,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), nullable\u003dTrue,"},{"line_number":115,"context_line":"                     index\u003dTrue)"},{"line_number":116,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), nullable\u003dTrue,"},{"line_number":117,"context_line":"                        index\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzRc%3D","line":114,"updated":"2013-07-11 07:46:45.000000000","message":"Extra FK.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5fe09e3250ae32286a58781477bf0da7112ce94","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    counter_name \u003d Column(String(255), nullable\u003dTrue)"},{"line_number":113,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"},{"line_number":114,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), nullable\u003dTrue,"},{"line_number":115,"context_line":"                     index\u003dTrue)"},{"line_number":116,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), nullable\u003dTrue,"},{"line_number":117,"context_line":"                        index\u003dTrue)"},{"line_number":118,"context_line":"    resource_id \u003d Column(String(255), ForeignKey(\u0027resource.id\u0027), nullable\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzVA%3D","line":115,"updated":"2013-07-11 07:25:03.000000000","message":"Most RDBMS create intexes for FK columns automatically. Is this index created in migrations scripts explicitly? If not, you should remove this too.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":113,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"},{"line_number":114,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), nullable\u003dTrue,"},{"line_number":115,"context_line":"                     index\u003dTrue)"},{"line_number":116,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), nullable\u003dTrue,"},{"line_number":117,"context_line":"                        index\u003dTrue)"},{"line_number":118,"context_line":"    resource_id \u003d Column(String(255), ForeignKey(\u0027resource.id\u0027), nullable\u003dTrue,"},{"line_number":119,"context_line":"                         index\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzRY%3D","line":116,"updated":"2013-07-11 07:46:45.000000000","message":"Extra FK.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":115,"context_line":"                     index\u003dTrue)"},{"line_number":116,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), nullable\u003dTrue,"},{"line_number":117,"context_line":"                        index\u003dTrue)"},{"line_number":118,"context_line":"    resource_id \u003d Column(String(255), ForeignKey(\u0027resource.id\u0027), nullable\u003dTrue,"},{"line_number":119,"context_line":"                         index\u003dTrue)"},{"line_number":120,"context_line":"    resource_metadata \u003d Column(JSONEncodedDict, nullable\u003dTrue)"},{"line_number":121,"context_line":"    counter_type \u003d Column(String(255), nullable\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzRU%3D","line":118,"updated":"2013-07-11 07:46:45.000000000","message":"Extra FK.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5fe09e3250ae32286a58781477bf0da7112ce94","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"class Project(Base):"},{"line_number":139,"context_line":"    __tablename__ \u003d \u0027project\u0027"},{"line_number":140,"context_line":"    id \u003d Column(String(255), primary_key\u003dTrue, index\u003dTrue, nullable\u003dFalse)"},{"line_number":141,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"},{"line_number":142,"context_line":"    resources \u003d relationship(\"Resource\", backref\u003d\u0027project\u0027)"},{"line_number":143,"context_line":"    meters \u003d relationship(\"Meter\", backref\u003d\u0027project\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzWg%3D","line":140,"updated":"2013-07-11 07:25:03.000000000","message":"ditto","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5fe09e3250ae32286a58781477bf0da7112ce94","unresolved":false,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"class Resource(Base):"},{"line_number":147,"context_line":"    __tablename__ \u003d \u0027resource\u0027"},{"line_number":148,"context_line":"    id \u003d Column(String(255), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":149,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"},{"line_number":150,"context_line":"    resource_metadata \u003d Column(JSONEncodedDict, nullable\u003dTrue)"},{"line_number":151,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), nullable\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzWc%3D","line":148,"updated":"2013-07-11 07:25:03.000000000","message":"ditto","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    id \u003d Column(String(255), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":149,"context_line":"    sources \u003d relationship(\"Source\", secondary\u003dlambda: sourceassoc)"},{"line_number":150,"context_line":"    resource_metadata \u003d Column(JSONEncodedDict, nullable\u003dTrue)"},{"line_number":151,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), nullable\u003dTrue,"},{"line_number":152,"context_line":"                     index\u003dTrue)"},{"line_number":153,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), nullable\u003dTrue,"},{"line_number":154,"context_line":"                        index\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzR0%3D","line":151,"updated":"2013-07-11 07:46:45.000000000","message":"Extra ForeignKey (missing in migrations).","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":150,"context_line":"    resource_metadata \u003d Column(JSONEncodedDict, nullable\u003dTrue)"},{"line_number":151,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), nullable\u003dTrue,"},{"line_number":152,"context_line":"                     index\u003dTrue)"},{"line_number":153,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), nullable\u003dTrue,"},{"line_number":154,"context_line":"                        index\u003dTrue)"},{"line_number":155,"context_line":"    meters \u003d relationship(\"Meter\", backref\u003d\u0027resource\u0027)"},{"line_number":156,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzRs%3D","line":153,"updated":"2013-07-11 07:46:45.000000000","message":"The same.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5fe09e3250ae32286a58781477bf0da7112ce94","unresolved":false,"context_lines":[{"line_number":158,"context_line":"class Alarm(Base):"},{"line_number":159,"context_line":"    \"\"\"Define Alarm data.\"\"\""},{"line_number":160,"context_line":"    __tablename__ \u003d \u0027alarm\u0027"},{"line_number":161,"context_line":"    id \u003d Column(String(255), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":162,"context_line":"    enabled \u003d Column(Boolean, nullable\u003dTrue)"},{"line_number":163,"context_line":"    name \u003d Column(Text, nullable\u003dTrue)"},{"line_number":164,"context_line":"    description \u003d Column(Text, nullable\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzWY%3D","line":161,"updated":"2013-07-11 07:25:03.000000000","message":"ditto","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":165,"context_line":"    timestamp \u003d Column(DateTime, default\u003dtimeutils.utcnow, nullable\u003dTrue)"},{"line_number":166,"context_line":"    counter_name \u003d Column(String(255), index\u003dTrue, nullable\u003dTrue)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), index\u003dTrue,"},{"line_number":169,"context_line":"                     nullable\u003dTrue)"},{"line_number":170,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), index\u003dTrue,"},{"line_number":171,"context_line":"                        nullable\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzUo%3D","line":168,"updated":"2013-07-11 07:46:45.000000000","message":"ForeignKey is skipped in migrations. Remove it or mark it as TODO for creation of migration.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"8c209f082a973245abaf4d08ad890fdea73ec245","unresolved":false,"context_lines":[{"line_number":165,"context_line":"    timestamp \u003d Column(DateTime, default\u003dtimeutils.utcnow, nullable\u003dTrue)"},{"line_number":166,"context_line":"    counter_name \u003d Column(String(255), index\u003dTrue, nullable\u003dTrue)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), index\u003dTrue,"},{"line_number":169,"context_line":"                     nullable\u003dTrue)"},{"line_number":170,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), index\u003dTrue,"},{"line_number":171,"context_line":"                        nullable\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzPc%3D","line":168,"in_reply_to":"AAAAP3%2F%2FzUo%3D","updated":"2013-07-11 07:50:34.000000000","message":"It does not work without ForeignKey.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    user_id \u003d Column(String(255), ForeignKey(\u0027user.id\u0027), index\u003dTrue,"},{"line_number":169,"context_line":"                     nullable\u003dTrue)"},{"line_number":170,"context_line":"    project_id \u003d Column(String(255), ForeignKey(\u0027project.id\u0027), index\u003dTrue,"},{"line_number":171,"context_line":"                        nullable\u003dTrue)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"    comparison_operator \u003d Column(String(2), nullable\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzUI%3D","line":170,"updated":"2013-07-11 07:46:45.000000000","message":"The same.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d5fe09e3250ae32286a58781477bf0da7112ce94","unresolved":false,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"class Event(Base):"},{"line_number":204,"context_line":"    __tablename__ \u003d \u0027event\u0027"},{"line_number":205,"context_line":"    id \u003d Column(Integer(11), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":206,"context_line":"    generated \u003d Column(Float(asdecimal\u003dTrue), index\u003dTrue, nullable\u003dTrue)"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"    unique_name_id \u003d Column(Integer(11), ForeignKey(\u0027unique_name.id\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzWU%3D","line":205,"updated":"2013-07-11 07:25:03.000000000","message":"ditto","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"a9ac6f85b217b09ff9c58aea9e49a3ee3e662c8e","unresolved":false,"context_lines":[{"line_number":205,"context_line":"    id \u003d Column(Integer(11), primary_key\u003dTrue, nullable\u003dFalse, index\u003dTrue)"},{"line_number":206,"context_line":"    generated \u003d Column(Float(asdecimal\u003dTrue), index\u003dTrue, nullable\u003dTrue)"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"    unique_name_id \u003d Column(Integer(11), ForeignKey(\u0027unique_name.id\u0027),"},{"line_number":209,"context_line":"                            index\u003dTrue, nullable\u003dTrue)"},{"line_number":210,"context_line":"    unique_name \u003d relationship(\"UniqueName\", backref\u003dbackref(\u0027unique_name\u0027,"},{"line_number":211,"context_line":"                               order_by\u003did))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2FzSo%3D","line":208,"updated":"2013-07-11 07:46:45.000000000","message":"Extra index\u003dTrue.","commit_id":"7d7c101a65a5f2467f68fb0d69434e20a58c9719"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"f8d7594faa5ba342dcdc20a0867e988c965ccbe8","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        Index(\u0027ix_meter_user_id\u0027, \u0027user_id\u0027),"},{"line_number":115,"context_line":"        Index(\u0027ix_meter_project_id\u0027, \u0027project_id\u0027),"},{"line_number":116,"context_line":"        Index(\u0027idx_meter_rid_cname\u0027, \u0027resource_id\u0027),"},{"line_number":117,"context_line":"        Index(\u0027idx_meter_rid_cname\u0027, \u0027counter_name\u0027),"},{"line_number":118,"context_line":"    )"},{"line_number":119,"context_line":"    id \u003d Column(Integer, primary_key\u003dTrue)"},{"line_number":120,"context_line":"    counter_name \u003d Column(String(255), nullable\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAP3%2F%2FzKg%3D","line":117,"updated":"2013-07-11 08:26:49.000000000","message":"Should be Index(\u0027idx_meter_rid_cname\u0027, \u0027resource_id\u0027, \u0027counter_name\u0027)","commit_id":"c28fb1f89e6ce3aa6f53a773121dbd1b30559c8b"},{"author":{"_account_id":6507,"name":"Shturm Svetlana","email":"svetlana.v.shturm@gmail.com","username":"sshturm"},"change_message_id":"f8d7594faa5ba342dcdc20a0867e988c965ccbe8","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    counter_unit \u003d Column(String(255), nullable\u003dTrue)"},{"line_number":128,"context_line":"    counter_volume \u003d Column(Float(53), nullable\u003dTrue)"},{"line_number":129,"context_line":"    timestamp \u003d Column(DateTime, default\u003dtimeutils.utcnow, nullable\u003dTrue,"},{"line_number":130,"context_line":"                       index\u003dTrue)"},{"line_number":131,"context_line":"    message_signature \u003d Column(String(1000), nullable\u003dTrue)"},{"line_number":132,"context_line":"    message_id \u003d Column(String(1000), nullable\u003dTrue)"},{"line_number":133,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAP3%2F%2FzKQ%3D","line":130,"updated":"2013-07-11 08:26:49.000000000","message":"Extra index\u003dTrue. It is defined in __table_args__ as `ix_meter_timestamp`","commit_id":"c28fb1f89e6ce3aa6f53a773121dbd1b30559c8b"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"9534739f866696248c8d6e80d5cc7322bd39c714","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    counter_volume \u003d Column(Float(53))"},{"line_number":128,"context_line":"    timestamp \u003d Column(DateTime, default\u003dtimeutils.utcnow)"},{"line_number":129,"context_line":"    message_signature \u003d Column(String(1000))"},{"line_number":130,"context_line":"    message_id \u003d Column(String(1000))"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"class User(Base):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAP3%2F%2FsbM%3D","line":130,"updated":"2013-07-12 21:23:39.000000000","message":"Mesage id values are UUIDs, so I don\u0027t think we need the column to be this big. I agree that this patch should sync the model to the migration, but it would be great if you could provide another patch later that changes the column size to something more reasonable for holding UUID values.","commit_id":"fe31a206aacc195403332a996f18a24711e63a59"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"d867032fb90d177e6808614a8ea3e3e184b40ef5","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    counter_volume \u003d Column(Float(53))"},{"line_number":128,"context_line":"    timestamp \u003d Column(DateTime, default\u003dtimeutils.utcnow)"},{"line_number":129,"context_line":"    message_signature \u003d Column(String(1000))"},{"line_number":130,"context_line":"    message_id \u003d Column(String(1000))"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"class User(Base):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAP3%2F%2FrTs%3D","line":130,"in_reply_to":"AAAAP3%2F%2FsbM%3D","updated":"2013-07-13 20:43:58.000000000","message":"sure but I think that this should be done after we add tests that checks that models and migrations are equal.","commit_id":"fe31a206aacc195403332a996f18a24711e63a59"}]}