)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"016e3a1ce297e4a09f1d8d282751a20ebc046b52","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"By default, Python uses a dict to store an object\u0027s instance attributes.The"},{"line_number":10,"context_line":"dict wastes a lot of RAM. Python can\u0027t just allocate a static amount of memory"},{"line_number":11,"context_line":"at object creation to store all the attributes. Therefore it sucks a lot of"},{"line_number":12,"context_line":"RAM if you create a lot of objects (I am talking in thousands and millions)."},{"line_number":13,"context_line":"I use__slots__ to tell Python not to use a dict, and only allocate space for"},{"line_number":14,"context_line":"a fixed set of attributes is tupe instead of dict."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_0468f4f4","line":12,"range":{"start_line":11,"start_character":48,"end_line":12,"end_character":75},"updated":"2016-11-15 02:36:02.000000000","message":"It would be better to mention some concrete numbers, that is some experiment results, how this patch saved memory usage with test environment information.","commit_id":"109509a7b557487dc33d97c081117e79b1c10359"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"016e3a1ce297e4a09f1d8d282751a20ebc046b52","unresolved":false,"context_lines":[{"line_number":11,"context_line":"at object creation to store all the attributes. Therefore it sucks a lot of"},{"line_number":12,"context_line":"RAM if you create a lot of objects (I am talking in thousands and millions)."},{"line_number":13,"context_line":"I use__slots__ to tell Python not to use a dict, and only allocate space for"},{"line_number":14,"context_line":"a fixed set of attributes is tupe instead of dict."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/driver-composition-reform.html#performance-impact"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_846a4419","line":14,"range":{"start_line":14,"start_character":29,"end_line":14,"end_character":33},"updated":"2016-11-15 02:36:02.000000000","message":"s/tupe/tuple/","commit_id":"109509a7b557487dc33d97c081117e79b1c10359"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"016e3a1ce297e4a09f1d8d282751a20ebc046b52","unresolved":false,"context_lines":[{"line_number":16,"context_line":"https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/driver-composition-reform.html#performance-impact"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ic7232f46c96599313011b0c7489b09cefbe93ae5"},{"line_number":19,"context_line":"Partial-Bug: #1524745"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a77a97e_c71a7e6e","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":21},"updated":"2016-11-15 02:36:02.000000000","message":"If this patch is for the driver-composition-reform, it would be better to explain the relation in the comment messages.","commit_id":"109509a7b557487dc33d97c081117e79b1c10359"}]}