/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 doc/developers/missing.txt

  • Committer: v.ladeuil+lp at free
  • Date: 2006-12-01 15:06:29 UTC
  • mto: (2172.3.1 bzr.73948)
  • mto: This revision was merged to the branch mainline in revision 2181.
  • Revision ID: v.ladeuil+lp@free.fr-20061201150629-zjd2an87u0r7nhhw
The tests that would have help avoid bug #73948 and all that mess :)

* bzrlib/transport/http/response.py:
(handle_response): Translate a 416 http error code into a bzr
exception.

* bzrlib/transport/http/_urllib2_wrappers.py:
(HTTPDefaultErrorHandler.http_error_default): Translate a 416 http
error code into a bzr exception.

* bzrlib/transport/http/_pycurl.py:
(PyCurlTransport._curl_perform): It could happen that pycrul
itself detect a short read.

* bzrlib/transport/http/__init__.py:
(HttpTransportBase._retry_get): New method, factorizing the retry
logic.
(HttpTransportBase.readv): We can have exception during the
initial GET worth degrading the range requirements (i.e. retrying
the GET request with either single or not ranges).

* bzrlib/tests/test_transport_implementations.py:
(TransportTests.test_readv_short_read): InvalidRange can also be
raised.

* bzrlib/tests/test_http.py:
(TestRangeRequestServer.test_readv_invalid_ranges): Was named
test_readv_short_read, the new name make the intent
clearer. Depending of the code path used (urllib or pycurl), both
exceptions can be raised.

* bzrlib/tests/HttpServer.py:
(TestingHTTPRequestHandler.do_GET): If invalid ranges are
specified, returns a 416 instead of the whole file (both are valid
according to the RFC).

Show diffs side-by-side

added added

removed removed

Lines of Context:
doc/developers/missing.txt
1
 
Missing
2
 
=======
3
 
 
4
 
Missing is used to find out the differences between the current branch and
5
 
another branch.
6
 
 
7
 
The performance analysis itself brings no further points than the
8
 
incremental-push-pull one.
9
 
 
10
 
More importantly, the UI have been considered not optimal: missing finds and
11
 
displays the differences between two branches, presenting the revisions that
12
 
are not common to both branches as two sets:
13
 
 
14
 
* the revisions that are present only in the current branch,
15
 
* the revisions that are present only in the other branch.
16
 
 
17
 
A quick and dirty survey indicates that most of the users are interested in
18
 
only one set of revisions at a time.
19
 
 
20
 
From a performance point of view, it may be more appropriate to calculate only
21
 
the set the user is asking for.
22
 
 
23
 
It has been proposed that the missing command be deprecated in favor of a
24
 
--dry-run option for the push, pull, merge commands.
25
 
 
26
 
In the mean time, the missing command stays interesting as it provides an easy
27
 
way to test, measure and optimize graph differences processing.