/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 tools/check-newsbugs.py

  • Committer: John Arbash Meinel
  • Date: 2009-10-29 16:15:43 UTC
  • mto: This revision was merged to the branch mainline in revision 4780.
  • Revision ID: john@arbash-meinel.com-20091029161543-tdqlm2l4e2z5o7le
We don't have to pad 'short' records.

When writing a row, we reserve 120 bytes from the first node so that we
can write our 'B+Tree Graph Index' signature and other meta-information.
For the root node, we don't always use the 120 bytes, and for non-root
rows, we don't use that data at all. So we usually pad back that
record. However, for indexes that fit entirely in the root record,
we don't pad them to 4096, and it turns out we don't need to pad
them with the spare 120 bytes either.

I was doing a test with lots of 'chained' btree indexes, and this
extra padding ended up being 4.6M => 4.3M of wasted space. I imagine
that bzr-search will have a similar issue with tiny indexes.

Show diffs side-by-side

added added

removed removed

Lines of Context:
tools/check-newsbugs.py
 
1
#!/usr/bin/python
 
2
# Simple script that will check which bugs mentioned in NEWS 
 
3
# are not yet marked Fix Released in Launchpad
 
4
 
 
5
import getopt, re, sys
 
6
try:
 
7
    from launchpadbugs import connector
 
8
except ImportError:
 
9
    print "Please install launchpadbugs from lp:python-launchpad-bugs"
 
10
    sys.exit(1)
 
11
 
 
12
options, args = getopt.gnu_getopt(sys.argv, "l", ["launchpad"])
 
13
options = dict(options)
 
14
 
 
15
if len(args) == 1:
 
16
    print "Usage: check-newsbugs [--launchpad] NEWS"
 
17
    print "Options:"
 
18
    print "--launchpad     Print out Launchpad mail commands for closing bugs "
 
19
    print "                that are already fixed."
 
20
    sys.exit(1)
 
21
 
 
22
 
 
23
def report_notmarked(bug, task, section):
 
24
    print 
 
25
    print "Bug %d was mentioned in NEWS but is not marked fix released:" % (bug.bugnumber, )
 
26
    print "Launchpad title: %s" % bug.title
 
27
    print "NEWS summary: "
 
28
    print section
 
29
    if "--launchpad" in options or "-l" in options:
 
30
        print "  bug %d" % bug.bugnumber
 
31
        print "  affects bzr"
 
32
        print "  status fixreleased"
 
33
 
 
34
 
 
35
def read_news_bugnos(path):
 
36
    """Read the bug numbers closed by a particular NEWS file
 
37
 
 
38
    :param path: Path to the NEWS file
 
39
    :return: list of bug numbers that were closed.
 
40
    """
 
41
    # Pattern to find bug numbers
 
42
    bug_pattern = re.compile("\#([0-9]+)")
 
43
    ret = set()
 
44
    f = open(path, 'r')
 
45
    try:
 
46
        section = ""
 
47
        for l in f.readlines():
 
48
            if l.strip() == "":
 
49
                try:
 
50
                    parenthesed = section.rsplit("(", 1)[1]
 
51
                except IndexError:
 
52
                    parenthesed = ""
 
53
                # Empty line, next section begins
 
54
                for bugno in [int(m) for m in bug_pattern.findall(parenthesed)]:
 
55
                    ret.add((bugno, section))
 
56
                section = ""
 
57
            else:
 
58
                section += l
 
59
        return ret
 
60
    finally:
 
61
        f.close()
 
62
 
 
63
open_bug = connector.ConnectBug("TEXT")
 
64
 
 
65
bugnos = read_news_bugnos(args[1])
 
66
for bugno, section in bugnos:
 
67
    bug = open_bug(url="https://bugs.launchpad.net/bzr/+bug/%d" % bugno)
 
68
    found_bzr = False
 
69
    for task in bug.infotable:
 
70
        if task.affects == "bzr":
 
71
            found_bzr = True
 
72
            if task.status != "Fix Released":
 
73
                report_notmarked(bug, task, section)
 
74
    if not found_bzr:
 
75
        print "Bug %d was mentioned in NEWS but is not marked as affecting bzr" % bugno