~bzr-pqm/bzr/bzr.dev

2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
1
"bzr update" performance analysis
2
=================================
3
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
4
There are 5 different slightly different situations in which bzr update 
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
5
can be used:
6
7
* local only (no-op)
8
* lightweight checkout
9
* heavy checkout
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
10
* heavy checkout w/ local changes
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
11
* bzr update could work on "bound branch" w/no wt
12
13
No new revisions
14
================
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
15
Should be O(1) to determine
16
Tree base is up to date
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
17
wt.last-rev == wt.b.last-rev
18
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
19
No local changes, only new revisions
20
====================================
21
1) Need to move wt.last_rev (O(1))
22
2) apply delta from base to new rev (O(changes))
2777.1.1 by Ian Clatworthy
(Ian Clatworthy)(trivial) Fix NEWS indenting and ReST nits in developers/update.txt
23
   applying changes to files is approx (O(lines-in-files ^ 2))
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
24
3) update meta-info (executable bits, etc) about modified files (O(changes))
25
26
2/3 could be concurrent (but that may not necessarily be faster)
27
28
potential issue w/ serialized is having 50k files in limbo/
29
30
the limbo/ directory could be avoided in some cases, for example when 
31
adding new files in new directories.
32
33
modifying in place: reduces fragmentation of fs, not atomic
34
w/ local modification, potential of data loss
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
35
w/o should be safe
36
37
"local mod" is diff between disk and last commit, not merge base
38
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
39
Detecting name conflicts should be O(siblings). Alternatively, conflicts 
2616.1.3 by Jelmer Vernooij
Fix typo, right complexity for applying changes to files.
40
with existing files can be detected using stat() and conflicts with new files 
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
41
can be detected by examining the pending transform. This changes 
42
complexity to O(changes).
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
43
44
out of date heavyweight checkout, out of date w/master
45
=======================================================
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
46
1) open working tree, check latest revision
47
2) open working tree branch, check latest revision
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
48
3) mismatch => update wt => wt.b.lastrev
2777.1.1 by Ian Clatworthy
(Ian Clatworthy)(trivial) Fix NEWS indenting and ReST nits in developers/update.txt
49
   apply delta to tree O(changed file size)
50
   ---- conflicts
51
   stop on conflicts
52
   stop always -> inform user they need to repeat (why not?, GFD)
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
53
4) pull new revs M => L O(newrevs)
54
5) apply delta to wt
2777.1.1 by Ian Clatworthy
(Ian Clatworthy)(trivial) Fix NEWS indenting and ReST nits in developers/update.txt
55
   local committed changes become a pending merge
56
   local uncommitted stay uncommitted
57
   local pending merges are retained (should be gc'd)
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
58
59
offtopic:
60
should bzr update report where the source is ?
61
should bzr update handle both cases (local tree out-of-date w/local branch, checkout out-of-date w/master) ?
62
63
if updating would diverge, give opportuniuty to branch/unbind instead
64
local ahead, "push to master"
65
66
ideas:
2616.1.2 by Jelmer Vernooij
Update after review by Aaron.
67
1) can this be done as a single logical step?
68
2) can this be done w/o modifying working tree until end? possible performance improvements
2616.1.1 by Jelmer Vernooij
Import performance analysis of update.
69
3) if the pulling revision step could deliver full texts, that may help for the merge (same thing as "bzr pull")