/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/benchmarks/bench_inventory.py

  • Committer: Vincent Ladeuil
  • Date: 2012-01-05 14:26:58 UTC
  • mto: This revision was merged to the branch mainline in revision 6468.
  • Revision ID: v.ladeuil+lp@free.fr-20120105142658-vek3v6pzlxb751s2
Tests passing for a first rough version of a cached branch config store. The changes here are too invasive and several parallel proposals have been made. 

@only_raises is evil and gave a hard time since any exception during
save_changes() was swallowed.

Possible improvements: 

- add some needs_write_lock decorators to crucial
  methods (_set_config_location ?) but keep locking the branch at higher levels

- decorate branch.unlock to call stack.save if last_lock() it True
  outside of @only_raises scope (evil decorator)

- add @needs_write_lock to stack.set and stack.remove (will probably get
  rid of most testing issues) we probably need a specialized decorator
  that can relay to the store and from there to the branch or whatever is
  needed. This will also helps bzr config to get it right. The
  get_mutable_section trick should not be needed anymore either.

- decorate branch.unlock to call stack.save if last_lock() it True outside
  of @only_raises scope (evil decorator)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/benchmarks/bench_inventory.py
1
 
# Copyright (C) 2006 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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
 
 
17
 
"""Tests for `bzrlib.inventory.Inventory` performance."""
18
 
 
19
 
 
20
 
import bzrlib.inventory
21
 
from bzrlib.benchmarks import Benchmark
22
 
 
23
 
 
24
 
class InvBenchmark(Benchmark):
25
 
    """Benchmarks for building large inventories."""
26
 
 
27
 
    def test_make_10824_inv_entries(self):
28
 
        """Making 10824 inv entries should be quick."""
29
 
        entries = []
30
 
        def make_10824_entries():
31
 
            for counter in xrange(10000):
32
 
                bzrlib.inventory.make_entry('file', 'foo',
33
 
                    "a_parent_id")
34
 
            for counter in xrange(824):
35
 
                bzrlib.inventory.make_entry('directory', 'foo',
36
 
                    "a_parent_id")
37
 
        # on roberts machine: this originally took:  533ms/  600ms
38
 
        # fixing slots to be vaguely accurate :      365ms/  419ms
39
 
        self.time(make_10824_entries)