← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/__init__.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2007-11-27 10:14:59 UTC
  • mfrom: (3034.1.1 trunk)
  • Revision ID: pqm@pqm.ubuntu.com-20071127101459-zcdf4aia34927kt0
Fix 165061 by using the correct _max_readv_combine attribute

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/transport/__init__.py
645
645
 
646
646
        :param relpath: The path to read data from.
647
647
        :param offsets: A list of (offset, size) tuples.
648
 
        :param adjust_for_latency: Adjust the requested offsets to accomdate
 
648
        :param adjust_for_latency: Adjust the requested offsets to accomodate
649
649
            transport latency. This may re-order the offsets, expand them to
650
650
            grab adjacent data when there is likely a high cost to requesting
651
651
            data relative to delivering it.
661
661
        if adjust_for_latency:
662
662
            # Design note: We may wish to have different algorithms for the
663
663
            # expansion of the offsets per-transport. E.g. for local disk to
664
 
            # use page-aligned expansion. If that is the case consider the following structure:
665
 
            #  - a test that transport.readv uses self._offset_expander or some similar attribute, to do the expansion
666
 
            #  - a test for each transport that it has some known-good offset expander
 
664
            # use page-aligned expansion. If that is the case consider the
 
665
            # following structure:
 
666
            #  - a test that transport.readv uses self._offset_expander or some
 
667
            #    similar attribute, to do the expansion
 
668
            #  - a test for each transport that it has some known-good offset
 
669
            #    expander
667
670
            #  - unit tests for each offset expander
668
671
            #  - a set of tests for the offset expander interface, giving
669
672
            #    baseline behaviour (which the current transport