Transport.*_multi functions should be removed

Bug #767228 reported by John A Meinel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Medium
Unassigned
Breezy
Fix Released
Medium
Jelmer Vernooij

Bug Description

A long long time ago, I had an idea about how to improve streaming performance without a smart server. We would allow the Transport.*_multi, etc functions take a generator, and yield a generator. Which would allow us to prefetch many files in one request. Thus things like "stat_multi" and "get_multi" were born.

However, we've decided that the only smart way forward is a real custom server protcol. And, in fact, only "Store.copy_multi" is used in real code, and IIRC Store is only used for the really-really-really old formats (pre weave).

They should be shot and buried, so that our test suite doesn't suffer for them.

Tags: easy tech-debt

Related branches

John A Meinel (jameinel)
tags: added: easy
Jelmer Vernooij (jelmer)
no longer affects: brz
Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
Jelmer Vernooij (jelmer)
tags: removed: check-for-breezy
Changed in brz:
status: New → Triaged
importance: Undecided → Medium
milestone: none → 3.0.0
assignee: nobody → Jelmer Vernooij (jelmer)
Jelmer Vernooij (jelmer)
Changed in brz:
status: Triaged → In Progress
Jelmer Vernooij (jelmer)
Changed in brz:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.