~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/medium.py

  • Committer: John Arbash Meinel
  • Date: 2008-09-05 02:29:34 UTC
  • mto: (3697.7.4 1.7)
  • mto: This revision was merged to the branch mainline in revision 3748.
  • Revision ID: john@arbash-meinel.com-20080905022934-s8692mbwpkdwi106
Cleanups to the algorithm documentation.

Show diffs side-by-side

added added

removed removed

Lines of Context:
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""The 'medium' layer for the smart servers and clients.
18
18
 
24
24
bzrlib/transport/smart/__init__.py.
25
25
"""
26
26
 
27
 
import errno
28
27
import os
29
28
import socket
30
29
import sys
32
31
 
33
32
from bzrlib.lazy_import import lazy_import
34
33
lazy_import(globals(), """
35
 
import atexit
36
 
import weakref
37
34
from bzrlib import (
38
 
    debug,
39
35
    errors,
 
36
    osutils,
40
37
    symbol_versioning,
41
 
    trace,
42
 
    ui,
43
38
    urlutils,
44
39
    )
45
 
from bzrlib.smart import client, protocol, request, vfs
 
40
from bzrlib.smart import protocol
46
41
from bzrlib.transport import ssh
47
42
""")
48
 
#usually already imported, and getting IllegalScoperReplacer on it here.
49
 
from bzrlib import osutils
50
 
 
51
 
# We must not read any more than 64k at a time so we don't risk "no buffer
52
 
# space available" errors on some platforms.  Windows in particular is likely
53
 
# to give error 10053 or 10055 if we read more than 64k from a socket.
54
 
_MAX_READ_SIZE = 64 * 1024
55
43
 
56
44
 
57
45
def _get_protocol_factory_for_bytes(bytes):
86
74
    return protocol_factory, bytes
87
75
 
88
76
 
89
 
def _get_line(read_bytes_func):
90
 
    """Read bytes using read_bytes_func until a newline byte.
91
 
 
92
 
    This isn't particularly efficient, so should only be used when the
93
 
    expected size of the line is quite short.
94
 
 
95
 
    :returns: a tuple of two strs: (line, excess)
96
 
    """
97
 
    newline_pos = -1
98
 
    bytes = ''
99
 
    while newline_pos == -1:
100
 
        new_bytes = read_bytes_func(1)
101
 
        bytes += new_bytes
102
 
        if new_bytes == '':
103
 
            # Ran out of bytes before receiving a complete line.
104
 
            return bytes, ''
105
 
        newline_pos = bytes.find('\n')
106
 
    line = bytes[:newline_pos+1]
107
 
    excess = bytes[newline_pos+1:]
108
 
    return line, excess
109
 
 
110
 
 
111
 
class SmartMedium(object):
112
 
    """Base class for smart protocol media, both client- and server-side."""
113
 
 
114
 
    def __init__(self):
115
 
        self._push_back_buffer = None
116
 
 
117
 
    def _push_back(self, bytes):
118
 
        """Return unused bytes to the medium, because they belong to the next
119
 
        request(s).
120
 
 
121
 
        This sets the _push_back_buffer to the given bytes.
122
 
        """
123
 
        if self._push_back_buffer is not None:
124
 
            raise AssertionError(
125
 
                "_push_back called when self._push_back_buffer is %r"
126
 
                % (self._push_back_buffer,))
127
 
        if bytes == '':
128
 
            return
129
 
        self._push_back_buffer = bytes
130
 
 
131
 
    def _get_push_back_buffer(self):
132
 
        if self._push_back_buffer == '':
133
 
            raise AssertionError(
134
 
                '%s._push_back_buffer should never be the empty string, '
135
 
                'which can be confused with EOF' % (self,))
136
 
        bytes = self._push_back_buffer
137
 
        self._push_back_buffer = None
138
 
        return bytes
139
 
 
140
 
    def read_bytes(self, desired_count):
141
 
        """Read some bytes from this medium.
142
 
 
143
 
        :returns: some bytes, possibly more or less than the number requested
144
 
            in 'desired_count' depending on the medium.
145
 
        """
146
 
        if self._push_back_buffer is not None:
147
 
            return self._get_push_back_buffer()
148
 
        bytes_to_read = min(desired_count, _MAX_READ_SIZE)
149
 
        return self._read_bytes(bytes_to_read)
150
 
 
151
 
    def _read_bytes(self, count):
152
 
        raise NotImplementedError(self._read_bytes)
153
 
 
154
 
    def _get_line(self):
155
 
        """Read bytes from this request's response until a newline byte.
156
 
 
157
 
        This isn't particularly efficient, so should only be used when the
158
 
        expected size of the line is quite short.
159
 
 
160
 
        :returns: a string of bytes ending in a newline (byte 0x0A).
161
 
        """
162
 
        line, excess = _get_line(self.read_bytes)
163
 
        self._push_back(excess)
164
 
        return line
165
 
 
166
 
    def _report_activity(self, bytes, direction):
167
 
        """Notify that this medium has activity.
168
 
 
169
 
        Implementations should call this from all methods that actually do IO.
170
 
        Be careful that it's not called twice, if one method is implemented on
171
 
        top of another.
172
 
 
173
 
        :param bytes: Number of bytes read or written.
174
 
        :param direction: 'read' or 'write' or None.
175
 
        """
176
 
        ui.ui_factory.report_transport_activity(self, bytes, direction)
177
 
 
178
 
 
179
 
class SmartServerStreamMedium(SmartMedium):
 
77
class SmartServerStreamMedium(object):
180
78
    """Handles smart commands coming over a stream.
181
79
 
182
80
    The stream may be a pipe connected to sshd, or a tcp socket, or an
185
83
    One instance is created for each connected client; it can serve multiple
186
84
    requests in the lifetime of the connection.
187
85
 
188
 
    The server passes requests through to an underlying backing transport,
 
86
    The server passes requests through to an underlying backing transport, 
189
87
    which will typically be a LocalTransport looking at the server's filesystem.
190
88
 
191
89
    :ivar _push_back_buffer: a str of bytes that have been read from the stream
203
101
        self.backing_transport = backing_transport
204
102
        self.root_client_path = root_client_path
205
103
        self.finished = False
206
 
        SmartMedium.__init__(self)
 
104
        self._push_back_buffer = None
 
105
 
 
106
    def _push_back(self, bytes):
 
107
        """Return unused bytes to the medium, because they belong to the next
 
108
        request(s).
 
109
 
 
110
        This sets the _push_back_buffer to the given bytes.
 
111
        """
 
112
        if self._push_back_buffer is not None:
 
113
            raise AssertionError(
 
114
                "_push_back called when self._push_back_buffer is %r"
 
115
                % (self._push_back_buffer,))
 
116
        if bytes == '':
 
117
            return
 
118
        self._push_back_buffer = bytes
 
119
 
 
120
    def _get_push_back_buffer(self):
 
121
        if self._push_back_buffer == '':
 
122
            raise AssertionError(
 
123
                '%s._push_back_buffer should never be the empty string, '
 
124
                'which can be confused with EOF' % (self,))
 
125
        bytes = self._push_back_buffer
 
126
        self._push_back_buffer = None
 
127
        return bytes
207
128
 
208
129
    def serve(self):
209
130
        """Serve requests until the client disconnects."""
236
157
 
237
158
    def _serve_one_request(self, protocol):
238
159
        """Read one request from input, process, send back a response.
239
 
 
 
160
        
240
161
        :param protocol: a SmartServerRequestProtocol.
241
162
        """
242
163
        try:
250
171
        """Called when an unhandled exception from the protocol occurs."""
251
172
        raise NotImplementedError(self.terminate_due_to_error)
252
173
 
253
 
    def _read_bytes(self, desired_count):
 
174
    def _get_bytes(self, desired_count):
254
175
        """Get some bytes from the medium.
255
176
 
256
177
        :param desired_count: number of bytes we want to read.
257
178
        """
258
 
        raise NotImplementedError(self._read_bytes)
259
 
 
 
179
        raise NotImplementedError(self._get_bytes)
 
180
 
 
181
    def _get_line(self):
 
182
        """Read bytes from this request's response until a newline byte.
 
183
        
 
184
        This isn't particularly efficient, so should only be used when the
 
185
        expected size of the line is quite short.
 
186
 
 
187
        :returns: a string of bytes ending in a newline (byte 0x0A).
 
188
        """
 
189
        newline_pos = -1
 
190
        bytes = ''
 
191
        while newline_pos == -1:
 
192
            new_bytes = self._get_bytes(1)
 
193
            bytes += new_bytes
 
194
            if new_bytes == '':
 
195
                # Ran out of bytes before receiving a complete line.
 
196
                return bytes
 
197
            newline_pos = bytes.find('\n')
 
198
        line = bytes[:newline_pos+1]
 
199
        self._push_back(bytes[newline_pos+1:])
 
200
        return line
 
201
 
260
202
 
261
203
class SmartServerSocketStreamMedium(SmartServerStreamMedium):
262
204
 
273
215
 
274
216
    def _serve_one_request_unguarded(self, protocol):
275
217
        while protocol.next_read_size():
276
 
            # We can safely try to read large chunks.  If there is less data
277
 
            # than _MAX_READ_SIZE ready, the socket wil just return a short
278
 
            # read immediately rather than block.
279
 
            bytes = self.read_bytes(_MAX_READ_SIZE)
 
218
            bytes = self._get_bytes(4096)
280
219
            if bytes == '':
281
220
                self.finished = True
282
221
                return
283
222
            protocol.accept_bytes(bytes)
284
 
 
 
223
        
285
224
        self._push_back(protocol.unused_data)
286
225
 
287
 
    def _read_bytes(self, desired_count):
288
 
        return _read_bytes_from_socket(
289
 
            self.socket.recv, desired_count, self._report_activity)
290
 
 
 
226
    def _get_bytes(self, desired_count):
 
227
        if self._push_back_buffer is not None:
 
228
            return self._get_push_back_buffer()
 
229
        # We ignore the desired_count because on sockets it's more efficient to
 
230
        # read 4k at a time.
 
231
        return self.socket.recv(4096)
 
232
    
291
233
    def terminate_due_to_error(self):
292
234
        # TODO: This should log to a server log file, but no such thing
293
235
        # exists yet.  Andrew Bennetts 2006-09-29.
295
237
        self.finished = True
296
238
 
297
239
    def _write_out(self, bytes):
298
 
        osutils.send_all(self.socket, bytes, self._report_activity)
 
240
        osutils.send_all(self.socket, bytes)
299
241
 
300
242
 
301
243
class SmartServerPipeStreamMedium(SmartServerStreamMedium):
320
262
 
321
263
    def _serve_one_request_unguarded(self, protocol):
322
264
        while True:
323
 
            # We need to be careful not to read past the end of the current
324
 
            # request, or else the read from the pipe will block, so we use
325
 
            # protocol.next_read_size().
326
265
            bytes_to_read = protocol.next_read_size()
327
266
            if bytes_to_read == 0:
328
267
                # Finished serving this request.
329
268
                self._out.flush()
330
269
                return
331
 
            bytes = self.read_bytes(bytes_to_read)
 
270
            bytes = self._get_bytes(bytes_to_read)
332
271
            if bytes == '':
333
272
                # Connection has been closed.
334
273
                self.finished = True
336
275
                return
337
276
            protocol.accept_bytes(bytes)
338
277
 
339
 
    def _read_bytes(self, desired_count):
 
278
    def _get_bytes(self, desired_count):
 
279
        if self._push_back_buffer is not None:
 
280
            return self._get_push_back_buffer()
340
281
        return self._in.read(desired_count)
341
282
 
342
283
    def terminate_due_to_error(self):
362
303
    request.finished_reading()
363
304
 
364
305
    It is up to the individual SmartClientMedium whether multiple concurrent
365
 
    requests can exist. See SmartClientMedium.get_request to obtain instances
366
 
    of SmartClientMediumRequest, and the concrete Medium you are using for
 
306
    requests can exist. See SmartClientMedium.get_request to obtain instances 
 
307
    of SmartClientMediumRequest, and the concrete Medium you are using for 
367
308
    details on concurrency and pipelining.
368
309
    """
369
310
 
378
319
    def accept_bytes(self, bytes):
379
320
        """Accept bytes for inclusion in this request.
380
321
 
381
 
        This method may not be called after finished_writing() has been
 
322
        This method may not be be called after finished_writing() has been
382
323
        called.  It depends upon the Medium whether or not the bytes will be
383
324
        immediately transmitted. Message based Mediums will tend to buffer the
384
325
        bytes until finished_writing() is called.
415
356
    def _finished_reading(self):
416
357
        """Helper for finished_reading.
417
358
 
418
 
        finished_reading checks the state of the request to determine if
 
359
        finished_reading checks the state of the request to determine if 
419
360
        finished_reading is allowed, and if it is hands off to _finished_reading
420
361
        to perform the action.
421
362
        """
435
376
    def _finished_writing(self):
436
377
        """Helper for finished_writing.
437
378
 
438
 
        finished_writing checks the state of the request to determine if
 
379
        finished_writing checks the state of the request to determine if 
439
380
        finished_writing is allowed, and if it is hands off to _finished_writing
440
381
        to perform the action.
441
382
        """
456
397
        return self._read_bytes(count)
457
398
 
458
399
    def _read_bytes(self, count):
459
 
        """Helper for SmartClientMediumRequest.read_bytes.
 
400
        """Helper for read_bytes.
460
401
 
461
402
        read_bytes checks the state of the request to determing if bytes
462
403
        should be read. After that it hands off to _read_bytes to do the
463
404
        actual read.
464
 
 
465
 
        By default this forwards to self._medium.read_bytes because we are
466
 
        operating on the medium's stream.
467
405
        """
468
 
        return self._medium.read_bytes(count)
 
406
        raise NotImplementedError(self._read_bytes)
469
407
 
470
408
    def read_line(self):
471
 
        line = self._read_line()
472
 
        if not line.endswith('\n'):
473
 
            # end of file encountered reading from server
474
 
            raise errors.ConnectionReset(
475
 
                "Unexpected end of message. Please check connectivity "
476
 
                "and permissions, and report a bug if problems persist.")
 
409
        """Read bytes from this request's response until a newline byte.
 
410
        
 
411
        This isn't particularly efficient, so should only be used when the
 
412
        expected size of the line is quite short.
 
413
 
 
414
        :returns: a string of bytes ending in a newline (byte 0x0A).
 
415
        """
 
416
        # XXX: this duplicates SmartClientRequestProtocolOne._recv_tuple
 
417
        line = ''
 
418
        while not line or line[-1] != '\n':
 
419
            new_char = self.read_bytes(1)
 
420
            line += new_char
 
421
            if new_char == '':
 
422
                # end of file encountered reading from server
 
423
                raise errors.ConnectionReset(
 
424
                    "please check connectivity and permissions",
 
425
                    "(and try -Dhpss if further diagnosis is required)")
477
426
        return line
478
427
 
479
 
    def _read_line(self):
480
 
        """Helper for SmartClientMediumRequest.read_line.
481
 
 
482
 
        By default this forwards to self._medium._get_line because we are
483
 
        operating on the medium's stream.
484
 
        """
485
 
        return self._medium._get_line()
486
 
 
487
 
 
488
 
class _DebugCounter(object):
489
 
    """An object that counts the HPSS calls made to each client medium.
490
 
 
491
 
    When a medium is garbage-collected, or failing that when atexit functions
492
 
    are run, the total number of calls made on that medium are reported via
493
 
    trace.note.
494
 
    """
495
 
 
496
 
    def __init__(self):
497
 
        self.counts = weakref.WeakKeyDictionary()
498
 
        client._SmartClient.hooks.install_named_hook(
499
 
            'call', self.increment_call_count, 'hpss call counter')
500
 
        atexit.register(self.flush_all)
501
 
 
502
 
    def track(self, medium):
503
 
        """Start tracking calls made to a medium.
504
 
 
505
 
        This only keeps a weakref to the medium, so shouldn't affect the
506
 
        medium's lifetime.
507
 
        """
508
 
        medium_repr = repr(medium)
509
 
        # Add this medium to the WeakKeyDictionary
510
 
        self.counts[medium] = dict(count=0, vfs_count=0,
511
 
                                   medium_repr=medium_repr)
512
 
        # Weakref callbacks are fired in reverse order of their association
513
 
        # with the referenced object.  So we add a weakref *after* adding to
514
 
        # the WeakKeyDict so that we can report the value from it before the
515
 
        # entry is removed by the WeakKeyDict's own callback.
516
 
        ref = weakref.ref(medium, self.done)
517
 
 
518
 
    def increment_call_count(self, params):
519
 
        # Increment the count in the WeakKeyDictionary
520
 
        value = self.counts[params.medium]
521
 
        value['count'] += 1
522
 
        try:
523
 
            request_method = request.request_handlers.get(params.method)
524
 
        except KeyError:
525
 
            # A method we don't know about doesn't count as a VFS method.
526
 
            return
527
 
        if issubclass(request_method, vfs.VfsRequest):
528
 
            value['vfs_count'] += 1
529
 
 
530
 
    def done(self, ref):
531
 
        value = self.counts[ref]
532
 
        count, vfs_count, medium_repr = (
533
 
            value['count'], value['vfs_count'], value['medium_repr'])
534
 
        # In case this callback is invoked for the same ref twice (by the
535
 
        # weakref callback and by the atexit function), set the call count back
536
 
        # to 0 so this item won't be reported twice.
537
 
        value['count'] = 0
538
 
        value['vfs_count'] = 0
539
 
        if count != 0:
540
 
            trace.note('HPSS calls: %d (%d vfs) %s',
541
 
                       count, vfs_count, medium_repr)
542
 
 
543
 
    def flush_all(self):
544
 
        for ref in list(self.counts.keys()):
545
 
            self.done(ref)
546
 
 
547
 
_debug_counter = None
548
 
 
549
 
 
550
 
class SmartClientMedium(SmartMedium):
 
428
 
 
429
class SmartClientMedium(object):
551
430
    """Smart client is a medium for sending smart protocol requests over."""
552
431
 
553
432
    def __init__(self, base):
561
440
        # _remote_version_is_before tracks the bzr version the remote side
562
441
        # can be based on what we've seen so far.
563
442
        self._remote_version_is_before = None
564
 
        # Install debug hook function if debug flag is set.
565
 
        if 'hpss' in debug.debug_flags:
566
 
            global _debug_counter
567
 
            if _debug_counter is None:
568
 
                _debug_counter = _DebugCounter()
569
 
            _debug_counter.track(self)
570
443
 
571
444
    def _is_remote_before(self, version_tuple):
572
445
        """Is it possible the remote side supports RPCs for a given version?
597
470
        """
598
471
        if (self._remote_version_is_before is not None and
599
472
            version_tuple > self._remote_version_is_before):
600
 
            # We have been told that the remote side is older than some version
601
 
            # which is newer than a previously supplied older-than version.
602
 
            # This indicates that some smart verb call is not guarded
603
 
            # appropriately (it should simply not have been tried).
604
473
            raise AssertionError(
605
474
                "_remember_remote_is_before(%r) called, but "
606
475
                "_remember_remote_is_before(%r) was called previously."
644
513
 
645
514
    def disconnect(self):
646
515
        """If this medium maintains a persistent connection, close it.
647
 
 
 
516
        
648
517
        The default implementation does nothing.
649
518
        """
650
 
 
 
519
        
651
520
    def remote_path_from_transport(self, transport):
652
521
        """Convert transport into a path suitable for using in a request.
653
 
 
 
522
        
654
523
        Note that the resulting remote path doesn't encode the host name or
655
524
        anything but path, so it is only safe to use it in requests sent over
656
525
        the medium from the matching transport.
684
553
 
685
554
    def _flush(self):
686
555
        """Flush the output stream.
687
 
 
 
556
        
688
557
        This method is used by the SmartClientStreamMediumRequest to ensure that
689
558
        all data for a request is sent, to avoid long timeouts or deadlocks.
690
559
        """
698
567
        """
699
568
        return SmartClientStreamMediumRequest(self)
700
569
 
 
570
    def read_bytes(self, count):
 
571
        return self._read_bytes(count)
 
572
 
701
573
 
702
574
class SmartSimplePipesClientMedium(SmartClientStreamMedium):
703
575
    """A client medium using simple pipes.
704
 
 
 
576
    
705
577
    This client does not manage the pipes: it assumes they will always be open.
706
578
    """
707
579
 
713
585
    def _accept_bytes(self, bytes):
714
586
        """See SmartClientStreamMedium.accept_bytes."""
715
587
        self._writeable_pipe.write(bytes)
716
 
        self._report_activity(len(bytes), 'write')
717
588
 
718
589
    def _flush(self):
719
590
        """See SmartClientStreamMedium._flush()."""
721
592
 
722
593
    def _read_bytes(self, count):
723
594
        """See SmartClientStreamMedium._read_bytes."""
724
 
        bytes = self._readable_pipe.read(count)
725
 
        self._report_activity(len(bytes), 'read')
726
 
        return bytes
 
595
        return self._readable_pipe.read(count)
727
596
 
728
597
 
729
598
class SmartSSHClientMedium(SmartClientStreamMedium):
730
599
    """A client medium using SSH."""
731
 
 
 
600
    
732
601
    def __init__(self, host, port=None, username=None, password=None,
733
602
            base=None, vendor=None, bzr_remote_path=None):
734
603
        """Creates a client that will connect on the first use.
735
 
 
 
604
        
736
605
        :param vendor: An optional override for the ssh vendor to use. See
737
606
            bzrlib.transport.ssh for details on ssh vendors.
738
607
        """
 
608
        SmartClientStreamMedium.__init__(self, base)
739
609
        self._connected = False
740
610
        self._host = host
741
611
        self._password = password
742
612
        self._port = port
743
613
        self._username = username
744
 
        # SmartClientStreamMedium stores the repr of this object in its
745
 
        # _DebugCounter so we have to store all the values used in our repr
746
 
        # method before calling the super init.
747
 
        SmartClientStreamMedium.__init__(self, base)
748
614
        self._read_from = None
749
615
        self._ssh_connection = None
750
616
        self._vendor = vendor
751
617
        self._write_to = None
752
618
        self._bzr_remote_path = bzr_remote_path
753
 
        # for the benefit of progress making a short description of this
754
 
        # transport
755
 
        self._scheme = 'bzr+ssh'
756
 
 
757
 
    def __repr__(self):
758
 
        return "%s(connected=%r, username=%r, host=%r, port=%r)" % (
759
 
            self.__class__.__name__,
760
 
            self._connected,
761
 
            self._username,
762
 
            self._host,
763
 
            self._port)
 
619
        if self._bzr_remote_path is None:
 
620
            symbol_versioning.warn(
 
621
                'bzr_remote_path is required as of bzr 0.92',
 
622
                DeprecationWarning, stacklevel=2)
 
623
            self._bzr_remote_path = os.environ.get('BZR_REMOTE_PATH', 'bzr')
764
624
 
765
625
    def _accept_bytes(self, bytes):
766
626
        """See SmartClientStreamMedium.accept_bytes."""
767
627
        self._ensure_connection()
768
628
        self._write_to.write(bytes)
769
 
        self._report_activity(len(bytes), 'write')
770
629
 
771
630
    def disconnect(self):
772
631
        """See SmartClientMedium.disconnect()."""
801
660
        """See SmartClientStreamMedium.read_bytes."""
802
661
        if not self._connected:
803
662
            raise errors.MediumNotConnected(self)
804
 
        bytes_to_read = min(count, _MAX_READ_SIZE)
805
 
        bytes = self._read_from.read(bytes_to_read)
806
 
        self._report_activity(len(bytes), 'read')
807
 
        return bytes
 
663
        return self._read_from.read(count)
808
664
 
809
665
 
810
666
# Port 4155 is the default port for bzr://, registered with IANA.
811
 
BZR_DEFAULT_INTERFACE = None
 
667
BZR_DEFAULT_INTERFACE = '0.0.0.0'
812
668
BZR_DEFAULT_PORT = 4155
813
669
 
814
670
 
815
671
class SmartTCPClientMedium(SmartClientStreamMedium):
816
672
    """A client medium using TCP."""
817
 
 
 
673
    
818
674
    def __init__(self, host, port, base):
819
675
        """Creates a client that will connect on the first use."""
820
676
        SmartClientStreamMedium.__init__(self, base)
826
682
    def _accept_bytes(self, bytes):
827
683
        """See SmartClientMedium.accept_bytes."""
828
684
        self._ensure_connection()
829
 
        osutils.send_all(self._socket, bytes, self._report_activity)
 
685
        osutils.send_all(self._socket, bytes)
830
686
 
831
687
    def disconnect(self):
832
688
        """See SmartClientMedium.disconnect()."""
840
696
        """Connect this medium if not already connected."""
841
697
        if self._connected:
842
698
            return
 
699
        self._socket = socket.socket()
 
700
        self._socket.setsockopt(socket.IPPROTO_TCP, socket.TCP_NODELAY, 1)
843
701
        if self._port is None:
844
702
            port = BZR_DEFAULT_PORT
845
703
        else:
846
704
            port = int(self._port)
847
705
        try:
848
 
            sockaddrs = socket.getaddrinfo(self._host, port, socket.AF_UNSPEC,
849
 
                socket.SOCK_STREAM, 0, 0)
850
 
        except socket.gaierror, (err_num, err_msg):
851
 
            raise errors.ConnectionError("failed to lookup %s:%d: %s" %
852
 
                    (self._host, port, err_msg))
853
 
        # Initialize err in case there are no addresses returned:
854
 
        err = socket.error("no address found for %s" % self._host)
855
 
        for (family, socktype, proto, canonname, sockaddr) in sockaddrs:
856
 
            try:
857
 
                self._socket = socket.socket(family, socktype, proto)
858
 
                self._socket.setsockopt(socket.IPPROTO_TCP,
859
 
                                        socket.TCP_NODELAY, 1)
860
 
                self._socket.connect(sockaddr)
861
 
            except socket.error, err:
862
 
                if self._socket is not None:
863
 
                    self._socket.close()
864
 
                self._socket = None
865
 
                continue
866
 
            break
867
 
        if self._socket is None:
 
706
            self._socket.connect((self._host, port))
 
707
        except socket.error, err:
868
708
            # socket errors either have a (string) or (errno, string) as their
869
709
            # args.
870
710
            if type(err.args) is str:
877
717
 
878
718
    def _flush(self):
879
719
        """See SmartClientStreamMedium._flush().
880
 
 
881
 
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and
 
720
        
 
721
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and 
882
722
        add a means to do a flush, but that can be done in the future.
883
723
        """
884
724
 
886
726
        """See SmartClientMedium.read_bytes."""
887
727
        if not self._connected:
888
728
            raise errors.MediumNotConnected(self)
889
 
        return _read_bytes_from_socket(
890
 
            self._socket.recv, count, self._report_activity)
 
729
        return self._socket.recv(count)
891
730
 
892
731
 
893
732
class SmartClientStreamMediumRequest(SmartClientMediumRequest):
906
745
 
907
746
    def _accept_bytes(self, bytes):
908
747
        """See SmartClientMediumRequest._accept_bytes.
909
 
 
 
748
        
910
749
        This forwards to self._medium._accept_bytes because we are operating
911
750
        on the mediums stream.
912
751
        """
915
754
    def _finished_reading(self):
916
755
        """See SmartClientMediumRequest._finished_reading.
917
756
 
918
 
        This clears the _current_request on self._medium to allow a new
 
757
        This clears the _current_request on self._medium to allow a new 
919
758
        request to be created.
920
759
        """
921
760
        if self._medium._current_request is not self:
922
761
            raise AssertionError()
923
762
        self._medium._current_request = None
924
 
 
 
763
        
925
764
    def _finished_writing(self):
926
765
        """See SmartClientMediumRequest._finished_writing.
927
766
 
929
768
        """
930
769
        self._medium._flush()
931
770
 
932
 
 
933
 
def _read_bytes_from_socket(sock, desired_count, report_activity):
934
 
    # We ignore the desired_count because on sockets it's more efficient to
935
 
    # read large chunks (of _MAX_READ_SIZE bytes) at a time.
936
 
    try:
937
 
        bytes = osutils.until_no_eintr(sock, _MAX_READ_SIZE)
938
 
    except socket.error, e:
939
 
        if len(e.args) and e.args[0] in (errno.ECONNRESET, 10054):
940
 
            # The connection was closed by the other side.  Callers expect an
941
 
            # empty string to signal end-of-stream.
942
 
            bytes = ''
943
 
        else:
944
 
            raise
945
 
    else:
946
 
        report_activity(len(bytes), 'read')
947
 
    return bytes
 
771
    def _read_bytes(self, count):
 
772
        """See SmartClientMediumRequest._read_bytes.
 
773
        
 
774
        This forwards to self._medium._read_bytes because we are operating
 
775
        on the mediums stream.
 
776
        """
 
777
        return self._medium._read_bytes(count)
948
778