)]}'
{"ceilometer/polling/manager.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"36ccb9d1e7c8598a01927f71a5b7c466a88272b9","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    cfg.IntOpt(\u0027polling_batch_size\u0027,"},{"line_number":52,"context_line":"               default\u003d50,"},{"line_number":53,"context_line":"               help\u003d\u0027Batch size of samples to send to notification agent, \u0027"},{"line_number":54,"context_line":"                    \u0027only acive when batch_polled_samples\u003dTrue\u0027),"},{"line_number":55,"context_line":"]"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"POLLING_OPTS \u003d ["}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_81c9d459","line":54,"updated":"2018-05-15 02:11:19.000000000","message":"s/acive/active/\n\nregardless, i think this would be better if this was just a single option. rather than having to have batched enabled and a number here.\n\nmaybe deprecate the previous option?","commit_id":"5aae79eaa06a4f799676daed0d44719c4c596dc7"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"36ccb9d1e7c8598a01927f71a5b7c466a88272b9","unresolved":false,"context_lines":[{"line_number":59,"context_line":"               default\u003d\"polling.yaml\","},{"line_number":60,"context_line":"               help\u003d\"Configuration file for polling definition.\""},{"line_number":61,"context_line":"               ),"},{"line_number":62,"context_line":"    cfg.StrOpt(\u0027partitioning_group_prefix\u0027,"},{"line_number":63,"context_line":"               deprecated_group\u003d\u0027central\u0027,"},{"line_number":64,"context_line":"               help\u003d\u0027Work-load partitioning group prefix. Use only if you \u0027"},{"line_number":65,"context_line":"                    \u0027want to run multiple polling agents with different \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_01e384da","line":62,"updated":"2018-05-15 02:11:19.000000000","message":"maybe add the option here instead so it\u0027s part of polling config group.","commit_id":"5aae79eaa06a4f799676daed0d44719c4c596dc7"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"18e1fa27460bf0a9cc6d1de8a76d000041408b54","unresolved":false,"context_lines":[{"line_number":209,"context_line":"                                sample, self._telemetry_secret"},{"line_number":210,"context_line":"                            ))"},{"line_number":211,"context_line":"                        if self._batch_size:"},{"line_number":212,"context_line":"                            if len(sample_batch) \u003e self._batch_size:"},{"line_number":213,"context_line":"                                self._send_notification(sample_batch)"},{"line_number":214,"context_line":"                                sample_batch \u003d []"},{"line_number":215,"context_line":"                            sample_batch.append(sample_dict)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5f7c97a3_01acbbea","line":212,"updated":"2018-05-16 14:41:43.000000000","message":"hm does this mean if we set batch_size \u003d 1, it push 2 at a time? i\u0027d prefer it to push (at most) what we define and set the min batch_size value to 1 like what we do on notification agent (https://github.com/openstack/ceilometer/blob/master/ceilometer/notification.py#L67)","commit_id":"fb99400ffd47e1106f38ea03c981691a78b0f2a1"}]}
