)]}'
{"test/unit/__init__.py":[{"author":{"_account_id":12261,"name":"Mahati","email":"mahati.chamarthy@gmail.com","username":"Mahati"},"change_message_id":"6744025d4b197a836a1a0f6eb08919a44c04d24e","unresolved":false,"context_lines":[{"line_number":1087,"context_line":"    consumed \u003d 0"},{"line_number":1088,"context_line":"    headers \u003d {}"},{"line_number":1089,"context_line":"    # First create a set of headers that consumes most, but not all, of the"},{"line_number":1090,"context_line":"    # meta_size budget. Use a fixed width of 8 for the header key suffix, which"},{"line_number":1091,"context_line":"    # is more than enough to represent the integer value. Each header will"},{"line_number":1092,"context_line":"    # therefore consume the key length of 8 plus MAX_META_VALUE_LENGTH from the"},{"line_number":1093,"context_line":"    # overall meta_size budget. (Note that the header prefix is not included in"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_2eda33ea","line":1090,"updated":"2016-08-02 12:47:38.000000000","message":"I\u0027m not sure on what basis are we assuming meta key suffix to be 8? Isn\u0027t it at user\u0027s discretion? Or do we have a predefined constraint for that as well?","commit_id":"8f4f46f1ae7ffa2ef7ad3458678c10cc1f30127f"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c21ca24ff58232064aec7e58fc51afc68f1fb3f9","unresolved":false,"context_lines":[{"line_number":1087,"context_line":"    consumed \u003d 0"},{"line_number":1088,"context_line":"    headers \u003d {}"},{"line_number":1089,"context_line":"    # First create a set of headers that consumes most, but not all, of the"},{"line_number":1090,"context_line":"    # meta_size budget. Use a fixed width of 8 for the header key suffix, which"},{"line_number":1091,"context_line":"    # is more than enough to represent the integer value. Each header will"},{"line_number":1092,"context_line":"    # therefore consume the key length of 8 plus MAX_META_VALUE_LENGTH from the"},{"line_number":1093,"context_line":"    # overall meta_size budget. (Note that the header prefix is not included in"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_31af322d","line":1090,"in_reply_to":"bacf61ea_2eda33ea","updated":"2016-08-02 13:03:46.000000000","message":"The suffix is what we add here, so its under our control. It\u0027s an integer, and we\u0027re making it a defined width field of 8.","commit_id":"8f4f46f1ae7ffa2ef7ad3458678c10cc1f30127f"},{"author":{"_account_id":12261,"name":"Mahati","email":"mahati.chamarthy@gmail.com","username":"Mahati"},"change_message_id":"a5e4c1d19a4f388805d353c38f772345a819e032","unresolved":false,"context_lines":[{"line_number":1087,"context_line":"    consumed \u003d 0"},{"line_number":1088,"context_line":"    headers \u003d {}"},{"line_number":1089,"context_line":"    # First create a set of headers that consumes most, but not all, of the"},{"line_number":1090,"context_line":"    # meta_size budget. Use a fixed width of 8 for the header key suffix, which"},{"line_number":1091,"context_line":"    # is more than enough to represent the integer value. Each header will"},{"line_number":1092,"context_line":"    # therefore consume the key length of 8 plus MAX_META_VALUE_LENGTH from the"},{"line_number":1093,"context_line":"    # overall meta_size budget. (Note that the header prefix is not included in"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_1ce32507","line":1090,"in_reply_to":"bacf61ea_31af322d","updated":"2016-08-02 13:10:18.000000000","message":"Ok, makes sense. I need some coffee. (I misread the comment above).","commit_id":"8f4f46f1ae7ffa2ef7ad3458678c10cc1f30127f"}],"test/unit/proxy/test_server.py":[{"author":{"_account_id":8542,"name":"Prashanth Pai","email":"ppai@redhat.com","username":"pai"},"change_message_id":"66452481cc227abfa3b146b07e396c5c628db91e","unresolved":false,"context_lines":[{"line_number":3458,"context_line":"        overall \u003d 0"},{"line_number":3459,"context_line":"        headers \u003d {}"},{"line_number":3460,"context_line":"        val \u003d \u0027x\u0027 * constraints.MAX_META_VALUE_LENGTH"},{"line_number":3461,"context_line":"        while overall + constraints.MAX_META_VALUE_LENGTH + 8 \u003c\u003d limit - 2:"},{"line_number":3462,"context_line":"            headers[\u0027X-Object-Meta-%8s\u0027 % overall] \u003d val"},{"line_number":3463,"context_line":"            overall +\u003d constraints.MAX_META_VALUE_LENGTH + 8"},{"line_number":3464,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_76c8b574","line":3461,"updated":"2016-08-01 14:45:02.000000000","message":"It took me a while to get why the 8 :)","commit_id":"7758b1f03ba839401fd0ee11f9556029f1313250"},{"author":{"_account_id":8542,"name":"Prashanth Pai","email":"ppai@redhat.com","username":"pai"},"change_message_id":"66452481cc227abfa3b146b07e396c5c628db91e","unresolved":false,"context_lines":[{"line_number":3464,"context_line":""},{"line_number":3465,"context_line":"        final_val \u003d \u0027x\u0027 * min("},{"line_number":3466,"context_line":"            limit - overall - 1, constraints.MAX_META_VALUE_LENGTH)"},{"line_number":3467,"context_line":""},{"line_number":3468,"context_line":"        test_headers \u003d dict(headers)"},{"line_number":3469,"context_line":"        test_headers_overall \u003d overall + len(final_val)"},{"line_number":3470,"context_line":"        key \u003d \u0027k\u0027 * (limit - test_headers_overall)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_f60e65e3","line":3467,"updated":"2016-08-01 14:45:02.000000000","message":"Some comments here about what\u0027s being done will help.","commit_id":"7758b1f03ba839401fd0ee11f9556029f1313250"},{"author":{"_account_id":12261,"name":"Mahati","email":"mahati.chamarthy@gmail.com","username":"Mahati"},"change_message_id":"87826519585276b646d30a44da62f8be4402aba0","unresolved":false,"context_lines":[{"line_number":3464,"context_line":""},{"line_number":3465,"context_line":"        final_val \u003d \u0027x\u0027 * min("},{"line_number":3466,"context_line":"            limit - overall - 1, constraints.MAX_META_VALUE_LENGTH)"},{"line_number":3467,"context_line":""},{"line_number":3468,"context_line":"        test_headers \u003d dict(headers)"},{"line_number":3469,"context_line":"        test_headers_overall \u003d overall + len(final_val)"},{"line_number":3470,"context_line":"        key \u003d \u0027k\u0027 * (limit - test_headers_overall)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_c82b9980","line":3467,"in_reply_to":"bacf61ea_f60e65e3","updated":"2016-08-02 08:58:28.000000000","message":"+1, a small note would be nice","commit_id":"7758b1f03ba839401fd0ee11f9556029f1313250"},{"author":{"_account_id":8542,"name":"Prashanth Pai","email":"ppai@redhat.com","username":"pai"},"change_message_id":"7c461e83040dc723dc0bc448d7c7ee94b12ef15e","unresolved":false,"context_lines":[{"line_number":3453,"context_line":"            dict((\u0027X-Object-Meta-%s\u0027 % i, i) for i in range(limit + 1)), 400)"},{"line_number":3454,"context_line":""},{"line_number":3455,"context_line":"        limit \u003d constraints.MAX_META_OVERALL_SIZE"},{"line_number":3456,"context_line":"        size, headers \u003d make_meta_headers_to_size(\u0027X-Object-Meta-\u0027, limit)"},{"line_number":3457,"context_line":"        self.assertEqual(limit, size)  # sanity check"},{"line_number":3458,"context_line":"        do_test(headers, 202)"},{"line_number":3459,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_0582a5c8","line":3456,"range":{"start_line":3456,"start_character":73,"end_line":3456,"end_character":74},"updated":"2016-08-04 11:21:49.000000000","message":"Neat!","commit_id":"8f4f46f1ae7ffa2ef7ad3458678c10cc1f30127f"}]}