~bzr-pqm/bzr/bzr.dev

2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
1
Incremental push/pull
2506.1.1 by Alexander Belchenko
sanitize developers docs
2
=====================
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
3
4
This use case covers pulling in or pushing out some number of revisions which
5
is typically a small fraction of the number already present in the target
6
repository. Pushing and pulling are defined as branch level operations for ease
7
of interaction with VCS systems that have no repository abstraction (such as
8
bzr-svn or GNU Arch) but within bzrlib's core they are currently the
9
responsibility of the Repository object.
10
11
Functional Requirements
2506.1.1 by Alexander Belchenko
sanitize developers docs
12
-----------------------
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
13
14
A push or pull operation must:
15
 * Copy all the data to reconstruct the selected revisions in the target
16
   branch. This is the goal of push and pull after all.
17
 * Reject corrupt data. As bzr has no innate mechanism for discarding corrupted
18
   data, corrupted data should not be incorporated accidentally.
19
20
Factors which should add work for push/pull
2506.1.1 by Alexander Belchenko
sanitize developers docs
21
-------------------------------------------
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
22
23
 * Baseline overhead: The time to connect to both branches.
24
 * Actual new data in the revisions being pulled (drives the amount of data to
25
   move around, includes the commit messages etc)
26
 * Number of revisions in the two repositories (scaling affects the
27
   determination of what revisions to move around).
28
29
Push/pull overview
2506.1.1 by Alexander Belchenko
sanitize developers docs
30
------------------
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
31
32
1. New data is identified in the source repository.
33
2. That data is read from the source repository.
34
3. The same data is verified and written to the target repository in such a
35
   manner that its not visible to readers until its ready for use.
36
37
New data identification
2506.1.1 by Alexander Belchenko
sanitize developers docs
38
~~~~~~~~~~~~~~~~~~~~~~~
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
39
40
We have a single top level data object: revisions. Everything else is
4031.3.1 by Frank Aspell
Fixing various typos
41
subordinate to revisions, so determining the revisions to propagate should be
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
42
all thats needed. This depends on revisions with partial data - such as those
43
with no signature - being flagged in some efficient manner.
44
45
We could do this in two manners: determine revisions to sync and signatures to sync in two passes, or change the 'value' of a revision implicitly when the signature is different. E.g. by using merkle hash trees with the signature data a separate component the signatures will naturally be identified to sync.
46
47
We want to only exchange data proportional to the number of new revisions and
48
signatures in the system though. One way to achieve this for revisions is to
49
walk the graph out from the desired tips until the surface area intersection is
50
found. For signatures a set difference seems to be needed as there is no DAG of signatures: the presence of one has no implications on the presence of another, so a full pass over the set of signatures would be required to confirm no new signatures are needed (let alone replaced signatures).
51
52
IFF we can determine 'new revisions' and 'new signatures' without full graph access then we can scale acceptable for push and pull.
53
54
Ghosts are revisions which are not present in a particular repository. Filling ghosts refers to removing ghosts in the target repository when the ghost is present in the source repository. Filling ghosts can be either an explicit or implicit action. The common case is no ghosts.
55
56
Set synchronisation approaches
57
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
58
59
A set synchronisation approach is one which synchronises two sets without
60
regard for innate structure. This can be very efficient but requires adding a
61
new node to be processed with every commit. Caching of the results of the
62
various set based syncs I've seen is possible but because the data structures
63
look different depending on the tip revision being synced up to the cache needs
64
to be very complex. I recommend not using such an approach for the common case
65
pull because of the failure to scale. We can use such an approach for
66
synchronisation of new signatures and ghosts, which should be an explicit
67
option in both cases.
68
69
DAG synchronisation approaches
70
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
71
72
A DAG based approach to synchronistion is one that uses the DAG structure to
73
determine the difference in present nodes. It can as a result operate from the
74
tip of the DAG backwards. A dag based approach should allow incremental access
75
to data and not require a full-graph scan for incremental operations.
76
77
File level scaling
78
^^^^^^^^^^^^^^^^^^
79
80
We should read roughly as much of the revision level graph as is needed from
81
each repository to determine the node difference.  If requested we should
82
perform a detailed scan to pick up ghost revisions and revisions which have had
83
signatures added. This should not be the default as it requires full history
84
access in both cases.
85
86
Expected file IO and access pattern:
87
88
 * Common case: repo with many branches of one project, to the same.
89
90
   1. Source and Target branch tips read.
91
   2. Find the tip of each branch in their repo (will require reading some of
92
      the revision graph but is typically near the end of the graph).
93
   3. Read and parse increasing amounts of the revision graph until one is
94
      found to be a subset of the other, or a complete list of revisions to be
95
      transmitted is created.
96
4853.1.1 by Patrick Regan
Removed trailing whitespace from files in doc directory
97
 * Uncommon cases:
98
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
99
   1. Repositories with many projects or branches which are very old may
100
      require reading a lot of unrelated graph data.
101
102
   1. Initial push/pull scenarios should not require reading an entire graph.
103
104
105
API scaling
106
^^^^^^^^^^^
107
108
 1. Get branch tips.
109
 2. Determine one sided graph difference. To avoid obtaining a full graph over
110
    the wire this needs to be done without reference to the full graph, and
111
    with some logarthmic scaling algorithm. There are several already available
4853.1.1 by Patrick Regan
Removed trailing whitespace from files in doc directory
112
    for this.
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
113
114
With ghost and new-signature detection:
115
116
 * File IO access pattern will read the entire graph on the 'target' side - if
117
   no ghosts are present then stop, otherwise seek the new revisions on the
118
   source side with the regular algorithm and also explicitly search for the
119
   ghost points from the target; plus a set difference search is needed on
120
   signatures.
121
122
 * Semantic level can probably be tuned, but as its also complex I suggest
123
   deferring analysis for optimal behaviour of this use case.
124
125
126
Data reading
2506.1.1 by Alexander Belchenko
sanitize developers docs
127
~~~~~~~~~~~~
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
128
129
When transferring information about a revision the graph of data for the
130
revision is walked: revision -> inventory, revision -> matching signature,
131
inventory -> file ids:revision pairs.
132
133
File level scaling
134
^^^^^^^^^^^^^^^^^^
135
136
As we're reading already committed data, as long as nothing is mutating data on
137
disk reading should be race free. We will:
138
139
 - read each revision object
140
 - read the matching inventory delta
141
 - attempt to read a signature object
142
 - parse the inventory delta
143
 - read the fileid:revisionid compressed chunk for each line in the inventory
144
   delta
145
146
Theres no point validating that the data read is valid, as transmission through to the client writing the data might invalidate it; we need to validate before we write.
147
148
API scaling
149
^^^^^^^^^^^
150
151
Given that we have established the revisions needed, a single API call should
152
suffice to obtain all data; the API should present the data in such an order
153
that it can be validated as it arrives and thus not require large scale
154
buffering on disk. Specifically each item of data should be validatable (e.g.
155
for some file data we want the fileid:revisionid:validationhash + content).
156
157
158
Data Verification and writing
2506.1.1 by Alexander Belchenko
sanitize developers docs
159
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
160
161
New data written to a repository should be completed intact when it is made
162
visible. This suggests that either all the data for a revision must be made
163
atomically visible (e.g. by renaming a single file) or the leaf nodes of the
164
reference graph must become visible first.
165
166
Data is referred to via the following graph:
167
revision -> revision
168
revision -> signature
169
revision -> inventory
170
inventory -> fileid:revisionid
171
fileid:revisionid -> fileid:revisionid
172
173
Data is verifiable via a different ordering:
174
signature -> revision -> inventory -> fileid:revisionid texts.
175
176
We dont gpg verify each revision today; this analysis only speaks to hash
177
verification of contents.
178
179
To validate a revision we need to validate the data it refers to. But to
180
validate the contents of a revision we need the new texts in the inventory for
181
the revision - to check a fileid:revisionid we need to know the expected sha1
182
of the full text and thus also need to read the delta chain to construct the
183
text as we accept it to determine if its valid. Providing separate validators
184
for the chosen representation would address this.
185
e.g: For an inventory entry FILEID:REVISIONID we store the validator of the
186
full text :SHA1:. If we also stored the validator of the chosen disk
187
representation (:DELTASHA1:) we could validate the transmitted representation
188
without expanding the delta in the common case. If that failed we could expand
189
the delta chain and try against the full text validator, and finally fail. As
190
different delta generators might generate different deltas, :DELTASHA1: should
191
not become part of the revision validator, only the inventory disk encoding. In
192
a related manner a transmission format that allowed cheap validation of content
193
without applying locally stored deltas would be advantageous because no local
194
reads would be incurred to validate new content. For instance, always sending a
195
full text for any file, possibly with a delta-chain when transmitting multiple
196
revisionids of the file, would allow this. (git pack-files have this property).
197
198
Overview summary
199
^^^^^^^^^^^^^^^^
200
201
A single-file local format would allow safe atomic addition of data while
202
allowing optimisal transmission order of data. Failing this the validation of
203
data should be tuned to not require reading local texts during data addition
204
even in the presence of delta chains. We should have transmission-validators
205
separate from content validators that allow validation of the delta-transmitted
206
form of objects.
207
208
File level scaling
209
^^^^^^^^^^^^^^^^^^
210
211
* Every new file text requires transmission and local serialisation.
212
* Every commit requires transmission and storage of a revision, signature and inventory.
213
214
Thus 4000 commits to a 50000 path tree of 10 files on averages requires (with
215
knits) between 26 writes (2*(3+10)) and 80006 (2*(4000*10 + 3)) writes. In all
216
cases there are 4000 * 13 distinct objects to record.
217
218
Grouping data by fileid, content and metadata, gives the figures above.
219
Data grouping:
220
221
* File per full identifier (fileid:revisionid:meta|content): 104000
4853.1.1 by Patrick Regan
Removed trailing whitespace from files in doc directory
222
* Delta-chain per object: object id count * constant overhead per object id
2481.1.5 by Robert Collins
Incremental push-pull performance anlysis draft.
223
  (26 -> 80006)
224
* Collation/pack file: 1
225
226
Performance for these depends heavily on implementation:
227
 - Using full ids we could name by validator or by id, giving best performance
228
   that depends on either receiving data in validator order or in id order.
229
 - using delta-chain per object we get least seek overhead and syscall overhead
230
   if we recieve in topological order within the object id, and object ids in
231
   lexical order.
232
 - Using a collation/pack file we can stream it into place and validate as we go,
233
   giving near ideal performance.
234
235
API scaling
236
^^^^^^^^^^^
237
238
The api for writing new data recieved over the network will need to be geared
239
to the transmission and local storage method. What we need is for the
240
transmission method to reasonably closely match the desired write ordering
241
locally. This suggests that once we decide on the best local storage means we
242
should design the api.
2485.4.5 by Robert Collins
Incremental push-pull notes.
243
244
245
take N commits from A to B, if B is local then merge changes into the tree.
246
copy ebough data to recreate snapshots
247
avoid ending up wth corrupt/bad data
248
2485.4.7 by Robert Collins
Review feedback.
249
Notes from London
2506.1.1 by Alexander Belchenko
sanitize developers docs
250
-----------------
2485.4.7 by Robert Collins
Review feedback.
251
252
 #. setup
253
4853.1.1 by Patrick Regan
Removed trailing whitespace from files in doc directory
254
   look at graph of revisions for ~N comits to deretmine eligibility for
2485.4.7 by Robert Collins
Review feedback.
255
   if preserve mainline is on, check LH only
2485.4.5 by Robert Collins
Incremental push-pull notes.
256
257
    identify objects to send that are not on the client repo
258
      - revision - may be proportional to the graph
259
      - inventory - proportional to work
260
      - texts     - proportional to work
261
      - signatures - ???
262
2485.4.7 by Robert Collins
Review feedback.
263
 #. data transmission
264
265
  * send data proportional to the new information
266
  * validate the data:
267
268
   #. validate the sha1 of the full text of each transmitted text.
269
   #. validate the sha1:name mapping in each newly referenced inventory item.
270
   #. validate the sha1 of the XML of each inventory against the revision.
2506.1.1 by Alexander Belchenko
sanitize developers docs
271
      **this is proportional to tree size and must be fixed**
2485.4.7 by Robert Collins
Review feedback.
272
273
 #. write the data to the local repo.
274
    The API should output the file texts needed by the merge as by product of the transmission
275
276
 #. tree application
277
278
Combine the output from the transmission step with additional 'new work data' for anything already in the local repository that is new in this tree.
2485.4.5 by Robert Collins
Incremental push-pull notes.
279
should write new files and stat existing files proportional to the count of the new work and the size of the full texts.