/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/counted_lock.py

  • Committer: Vincent Ladeuil
  • Date: 2007-12-08 23:15:18 UTC
  • mto: (2929.3.16 https) (3097.2.1 trunk)
  • mto: This revision was merged to the branch mainline in revision 3099.
  • Revision ID: v.ladeuil+lp@free.fr-20071208231518-sj2ui57xyd4mkjra
Limit GET requests by body size instead of number of ranges.

* bzrlib/transport/http/response.py:
Fix some error messages.
(RangeFile.read_range_definition): Keep the range headers in the
_headers attribute for easier debugging (especially for remote
debugging).

* bzrlib/transport/http/_pycurl.py:
(PyCurlTransport): Replace _max_readv_combine by _get_max_size
which is more approriate to the problem.

* bzrlib/transport/http/__init__.py:
(HttpTransportBase): Add a _get_max_size class attribute
corresponding to the max_size _coalesced_offets max_size
parameter.
(HttpTransportBase._coalesce_readv): Limit the size of the get
requests if _get_max_size is greater than 0 while still respecting
the maximum number of ranges in a request.

* bzrlib/tests/test_http.py:
(TestRangeRequestServer.test_readv_get_max_size): Test the
_get_max_size parameter.

* bzrlib/transport/__init__.py:
(Transport._coalesce_offsets): Add a max_size parameter limiting
the size of the coalesced offsets.

* bzrlib/tests/test_transport.py:
(TestCoalesceOffsets.check): Add the max_size parameter.
(TestCoalesceOffsets.test_coalesce_max_size,
TestCoalesceOffsets.test_coalesce_no_max_size): Test the max_size
parameter.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/counted_lock.py
 
1
# Copyright (C) 2007 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
"""Counted lock class"""
 
18
 
 
19
 
 
20
from bzrlib.errors import (
 
21
    LockError,
 
22
    ReadOnlyError,
 
23
    )
 
24
 
 
25
 
 
26
# TODO: Pass through lock tokens on lock_write and read, and return them...
 
27
#
 
28
# TODO: Allow upgrading read locks to write?  Conceptually difficult.
 
29
 
 
30
 
 
31
class CountedLock(object):
 
32
    """Decorator around a lock that makes it reentrant.
 
33
 
 
34
    This can be used with any object that provides a basic Lock interface,
 
35
    including LockDirs and OS file locks.
 
36
    """
 
37
 
 
38
    def __init__(self, real_lock):
 
39
        self._real_lock = real_lock
 
40
        self._lock_mode = None
 
41
        self._lock_count = 0
 
42
 
 
43
    def break_lock(self):
 
44
        self._real_lock.break_lock()
 
45
        self._lock_mode = None
 
46
        self._lock_count = 0
 
47
 
 
48
    def is_locked(self):
 
49
        return self._lock_mode is not None
 
50
 
 
51
    def lock_read(self):
 
52
        """Acquire the lock in read mode.
 
53
 
 
54
        If the lock is already held in either read or write mode this
 
55
        increments the count and succeeds.  If the lock is not already held,
 
56
        it is taken in read mode.
 
57
        """
 
58
        if self._lock_mode:
 
59
            assert self._lock_mode in ('r', 'w'), \
 
60
                   "invalid lock mode %r" % self._lock_mode
 
61
            self._lock_count += 1
 
62
        else:
 
63
            assert self._lock_count == 0
 
64
            self._real_lock.lock_read()
 
65
            self._lock_count = 1
 
66
            self._lock_mode = 'r'
 
67
 
 
68
    def lock_write(self):
 
69
        """Acquire the lock in write mode.
 
70
 
 
71
        If the lock was originally acquired in read mode this will fail.
 
72
        """
 
73
        if self._lock_count == 0:
 
74
            assert self._lock_mode is None
 
75
            self._real_lock.lock_write()
 
76
            self._lock_mode = 'w'
 
77
        elif self._lock_mode != 'w':
 
78
            raise ReadOnlyError(self)
 
79
        self._lock_count += 1
 
80
 
 
81
    def unlock(self):
 
82
        if self._lock_count == 0:
 
83
            raise LockError("%s not locked" % (self,))
 
84
        elif self._lock_count == 1:
 
85
            self._real_lock.unlock()
 
86
            self._lock_mode = None
 
87
        self._lock_count -= 1