Skip to content
This repository has been archived by the owner on Feb 9, 2022. It is now read-only.

Add support for report batching in Mixer #1475

Open
geeknoid opened this issue Oct 20, 2017 · 1 comment
Open

Add support for report batching in Mixer #1475

geeknoid opened this issue Oct 20, 2017 · 1 comment

Comments

@geeknoid
Copy link
Contributor

The interface for REPORT adapters supports receiving an array of instances at a time, but Mixer currently only delivers single items. Given that Mixer receives reports in batches, it would likely be more efficient for Mixer to process an incoming batch of reports as a whole and deliver that whole batch to the adapter en masse.

We should evaluate whether this would actually be a win. There will be transient memory allocations and other overheads associated in batching in Mixer. So it might not be a good thing overall to batch in Mixer like that. It's especially interesting when considering the adapters might want to batch things in a different way than Mixer does.

@geeknoid geeknoid added this to the Istio 0.3 milestone Oct 20, 2017
@geeknoid geeknoid changed the title Ad support for report batching in Mixer Add support for report batching in Mixer Oct 20, 2017
@geeknoid
Copy link
Contributor Author

See #849 for ideas on necessary config for this, and see #1501 about supporting generic per-adapter config.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants