)]}'
{"ceilometerclient/v2/shell.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"6707c5eb0a01684056e9cb6991e5489cb815c7ba","unresolved":false,"context_lines":[{"line_number":56,"context_line":"COMPLEX_OPERATORS \u003d [\u0027and\u0027, \u0027or\u0027]"},{"line_number":57,"context_line":"SIMPLE_OPERATORS \u003d [\"\u003d\", \"!\u003d\", \"\u003c\", \"\u003c\u003d\", \u0027\u003e\u0027, \u0027\u003e\u003d\u0027]"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"DEFAULT_API_LIMIT \u003d (\u0027\u003cdefault_api_return_limit\u003e; 100 if parameter was not \u0027"},{"line_number":60,"context_line":"                     \u0027configured\u0027)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a7e1126_f78b6766","line":59,"updated":"2015-12-18 15:25:50.000000000","message":"i don\u0027t think we should hardcode 100 since that number is configurable. we should point to the configuration option","commit_id":"2248b545119f6648bf741a1650f8797aadfbbda4"},{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"6f16267e8cd18d085aee02a348c2d6fd88807259","unresolved":false,"context_lines":[{"line_number":56,"context_line":"COMPLEX_OPERATORS \u003d [\u0027and\u0027, \u0027or\u0027]"},{"line_number":57,"context_line":"SIMPLE_OPERATORS \u003d [\"\u003d\", \"!\u003d\", \"\u003c\", \"\u003c\u003d\", \u0027\u003e\u0027, \u0027\u003e\u003d\u0027]"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"DEFAULT_API_LIMIT \u003d (\u0027\u003cdefault_api_return_limit\u003e; 100 if parameter was not \u0027"},{"line_number":60,"context_line":"                     \u0027configured\u0027)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a7e1126_12fc01b7","line":59,"in_reply_to":"3a7e1126_f78b6766","updated":"2015-12-18 15:42:29.000000000","message":"Done","commit_id":"2248b545119f6648bf741a1650f8797aadfbbda4"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"23d4838b30c9e77cac31cf9e68594385d4c803f9","unresolved":false,"context_lines":[{"line_number":137,"context_line":"@utils.arg(\u0027-m\u0027, \u0027--meter\u0027, metavar\u003d\u0027\u003cNAME\u003e\u0027,"},{"line_number":138,"context_line":"           action\u003dNotEmptyAction, help\u003d\u0027Name of meter to show samples for.\u0027)"},{"line_number":139,"context_line":"@utils.arg(\u0027-l\u0027, \u0027--limit\u0027, metavar\u003d\u0027\u003cNUMBER\u003e\u0027,"},{"line_number":140,"context_line":"           default\u003dDEFAULT_API_LIMIT,"},{"line_number":141,"context_line":"           help\u003d\u0027Maximum number of samples to return.\u0027)"},{"line_number":142,"context_line":"def do_sample_list(cc, args):"},{"line_number":143,"context_line":"    \"\"\"List the samples (return OldSample objects if -m/--meter is set).\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"3a7e1126_d6befb63","line":140,"updated":"2015-12-22 13:00:13.000000000","message":"i\u0027m not sure you can make this a value like that. i think you can only add it to help text","commit_id":"df3aa2bbb9eaa7df4b457757cfeb14fdfc9d1086"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"da8a46016be0460212fd48e44038d7292b5ea574","unresolved":false,"context_lines":[{"line_number":56,"context_line":"COMPLEX_OPERATORS \u003d [\u0027and\u0027, \u0027or\u0027]"},{"line_number":57,"context_line":"SIMPLE_OPERATORS \u003d [\"\u003d\", \"!\u003d\", \"\u003c\", \"\u003c\u003d\", \u0027\u003e\u0027, \u0027\u003e\u003d\u0027]"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"DEFAULT_API_LIMIT \u003d (\u0027It is mandatory parameter, default to \u0027"},{"line_number":60,"context_line":"                     \u0027\u003cdefault_api_return_limit\u003e which is configured in \u0027"},{"line_number":61,"context_line":"                     \u0027ceilometer.conf [api] group\u0027)"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a7b0d38_d1036475","line":59,"updated":"2015-12-22 15:26:55.000000000","message":"seems incorrect to say it\u0027s a mandatory parameter. i think it\u0027s safe to just say results default to option value if not param not provided.","commit_id":"9e374a33892b45759b34257f0aea85e3ea7edb18"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"7790a2af199c3a38a08bd91572faa34a9530cac0","unresolved":false,"context_lines":[{"line_number":56,"context_line":"COMPLEX_OPERATORS \u003d [\u0027and\u0027, \u0027or\u0027]"},{"line_number":57,"context_line":"SIMPLE_OPERATORS \u003d [\"\u003d\", \"!\u003d\", \"\u003c\", \"\u003c\u003d\", \u0027\u003e\u0027, \u0027\u003e\u003d\u0027]"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"DEFAULT_API_LIMIT \u003d (\u0027Results default to \u003cdefault_api_return_limit\u003e value if \u0027"},{"line_number":60,"context_line":"                     \u0027param is not provided. Option is configured in \u0027"},{"line_number":61,"context_line":"                     \u0027ceilometer.conf [api] group\u0027)"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"fa69d971_8542bc0d","line":59,"updated":"2016-01-11 20:24:30.000000000","message":"maybe say \"API server limits result to \u003cdefault_api_return_limit\u003e rows if no limit provided.\" so the user is aware it\u0027s a server setting and not a client setting.","commit_id":"e7de1fd095688f56bc7ba8c6ed8ae58195de994f"},{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"65beb57922591fbe45fcc80d7b574495e25602c0","unresolved":false,"context_lines":[{"line_number":56,"context_line":"COMPLEX_OPERATORS \u003d [\u0027and\u0027, \u0027or\u0027]"},{"line_number":57,"context_line":"SIMPLE_OPERATORS \u003d [\"\u003d\", \"!\u003d\", \"\u003c\", \"\u003c\u003d\", \u0027\u003e\u0027, \u0027\u003e\u003d\u0027]"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"DEFAULT_API_LIMIT \u003d (\u0027Results default to \u003cdefault_api_return_limit\u003e value if \u0027"},{"line_number":60,"context_line":"                     \u0027param is not provided. Option is configured in \u0027"},{"line_number":61,"context_line":"                     \u0027ceilometer.conf [api] group\u0027)"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"da6ed579_21abdd76","line":59,"in_reply_to":"fa69d971_8542bc0d","updated":"2016-01-14 15:05:30.000000000","message":"Done","commit_id":"e7de1fd095688f56bc7ba8c6ed8ae58195de994f"}]}
