[tahoe-lafs-trac-stream] [tahoe-lafs] #2119: Don't pre-convert all serverids to ServerTrackers

[tahoe-lafs]

#2119: Don't pre-convert all serverids to ServerTrackers
------------------------+---------------------------
 Reporter:  markberger  |          Owner:  daira
     Type:  defect      |         Status:  new
 Priority:  normal      |      Milestone:  undecided
Component:  unknown     |        Version:  1.10.0
 Keywords:              |  Launchpad Bug:
------------------------+---------------------------
 In `immutable/upload.py` we have the following code:

 {{{
     def _get_next_allocation(self):
         """
         Return the next share allocation that we need to make.

         Specifically, I return a tuple (tracker, shares_to_ask), where
         tracker is a ServerTracker instance and shares_to_ask is a set of
         shares that we should store on that server. If there are no more
         allocations to make, I return None.
         """

         while self.trackers:
             tracker = self.trackers.pop(0)
             # TODO: don't pre-convert all serverids to ServerTrackers
             assert isinstance(tracker, ServerTracker)
 }}}

 As part of #1382 I am removing the TODO comment and creating a ticket for
 it instead.

-- 
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/2119>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage