mimir icon indicating copy to clipboard operation
mimir copied to clipboard

Expose thanos object storage option to download / upload in parallel

Open 56quarters opened this issue 3 years ago • 5 comments
trafficstars

Is your feature request related to a problem? Please describe.

https://github.com/thanos-io/thanos/commit/79ab7c65cb4b66b9dcc4fa537cb43b00cc65066c

Thanos added the ability to download and upload blocks in parallel. This seems like a useful setting to make available (or set to some default value other than 1).

Describe the solution you'd like

Let this setting be configured to speed up compactor download and upload of TSDB blocks.

Describe alternatives you've considered

Nothing so far.

56quarters avatar Jul 14 '22 15:07 56quarters

Is this related? https://github.com/grafana/mimir/pull/552

pstibrany avatar Jul 14 '22 15:07 pstibrany

Is this related? #552

Yep, related but not exactly the same AFAICT. #552 downloads multiple blocks in parallel while the change referenced here would download each file that is part of a block in parallel. Given that, maybe it doesn't make sense to expose this setting. A benchmark would be required to tell if there's any benefit to doing this.

56quarters avatar Jul 14 '22 16:07 56quarters

I would like to first learn which Mimir component would benefit from it.

pracucci avatar Jul 15 '22 09:07 pracucci

I would like to first learn which Mimir component would benefit from it.

The compactor is currently the only component that calls the Thanos block.Download() method which is where this would apply.

56quarters avatar Jul 27 '22 14:07 56quarters

I would like to first learn which Mimir component would benefit from it.

The compactor is currently the only component that calls the Thanos block.Download() method which is where this would apply.

Mimir download multiple source blocks in parallel. Having a double parallelism could be harder to reason about in terms of max concurrency.

pracucci avatar Jul 28 '22 07:07 pracucci