##// END OF EJS Templates
inotify: server: new data structure to keep track of changes....
inotify: server: new data structure to keep track of changes. == Rationale for the new structure == Current structure was a dictionary tree. One directory was tracked as a dictionary: - keys: file/subdir name - values: - for a file, the status (a/r/m/...) - for a subdir, the directory representing the subdir It allowed efficient lookups, no matter of the type of the terminal leaf: for part in path.split('/'): tree = tree[part] However, there is no way to represent a directory and a file with the same name because keys are conflicting in the dictionary. Concrete example: Initial state: root dir |- foo (file) |- bar (file) # data state is: {'foo': 'n', 'bar': 'n'} Remove foo: root dir |- bar (file) # Data becomes {'foo': 'r'} until next commit. Add foo, as a directory, and foo/barbar file: root dir |- bar (file) |-> foo (dir) |- barbar (file) # New state should be represented as: {'foo': {'barbar': 'a'}, 'bar': 'n'} however, the key "foo" is already used and represents the old file. The dirstate: D foo A foo/barbar cannot be represented, hence the need for a new structure. == The new structure == 'directory' class. Represents one directory level. * Notable attributes: Two dictionaries: - 'files' Maps filename -> status for the current dir. - 'dirs' Maps subdir's name -> directory object representing the subdir * methods - walk(), formerly server.walk - lookup(), old server.lookup - dir(), old server.dir This new class allows embedding all the tree walks/lookups in its own class, instead of having everything mixed together in server. Incidently, since files and directories are not stored in the same dictionaries, we are solving the previous key conflict problem. The small drawback is that lookup operation is a bit more complex: for a path a/b/c/d/e we have to check twice the leaf, if e is a directory or a file.

File last commit:

r6338:0750f111 default
r9115:b55d4471 default
Show More
test-push-http.out
38 lines | 1.1 KiB | text/plain | TextLexer
/ tests / test-push-http.out
Vadim Gelfer
add tests for push over http.
r2481 adding a
Adrian Buehlmann
clone: print "updating working directory" status message...
r6338 updating working directory
Vadim Gelfer
add tests for push over http.
r2481 1 files updated, 0 files merged, 0 files removed, 0 files unresolved
% expect ssl error
Bryan O'Sullivan
Fix accidentally broken test.
r5386 pushing to http://localhost/
Vadim Gelfer
add tests for push over http.
r2481 searching for changes
ssl required
Dirkjan Ochtman
tests: easier hg serve error diagnosis
r6167 % serve errors
Vadim Gelfer
add tests for push over http.
r2481 % expect authorization error
Dirkjan Ochtman
hgweb: only accept POST requests for unbundle
r6335 abort: authorization failed
Bryan O'Sullivan
Fix accidentally broken test.
r5386 pushing to http://localhost/
Vadim Gelfer
add tests for push over http.
r2481 searching for changes
Dirkjan Ochtman
tests: easier hg serve error diagnosis
r6167 % serve errors
Vadim Gelfer
add tests for push over http.
r2481 % expect authorization error: must have authorized user
Dirkjan Ochtman
hgweb: only accept POST requests for unbundle
r6335 abort: authorization failed
Bryan O'Sullivan
Fix accidentally broken test.
r5386 pushing to http://localhost/
Vadim Gelfer
add tests for push over http.
r2481 searching for changes
Dirkjan Ochtman
tests: easier hg serve error diagnosis
r6167 % serve errors
Vadim Gelfer
add tests for push over http.
r2481 % expect success
Bryan O'Sullivan
Fix accidentally broken test.
r5386 pushing to http://localhost/
Vadim Gelfer
add tests for push over http.
r2481 searching for changes
adding changesets
adding manifests
adding file changes
added 1 changesets with 1 changes to 1 files
Dirkjan Ochtman
tests: easier hg serve error diagnosis
r6167 % serve errors
changegroup hook: HG_NODE=ba677d0156c1196c1a699fa53f390dcfc3ce3872 HG_SOURCE=serve HG_URL=remote:http
Vadim Gelfer
add tests for push over http.
r2481 rolling back last transaction
% expect authorization error: all users denied
Dirkjan Ochtman
hgweb: only accept POST requests for unbundle
r6335 abort: authorization failed
Bryan O'Sullivan
Allow tests to run in parallel.
r5384 pushing to http://localhost/
Vadim Gelfer
add tests for push over http.
r2481 searching for changes
Dirkjan Ochtman
tests: easier hg serve error diagnosis
r6167 % serve errors
Vadim Gelfer
add tests for push over http.
r2481 % expect authorization error: some users denied, users must be authenticated
Dirkjan Ochtman
hgweb: only accept POST requests for unbundle
r6335 abort: authorization failed
Bryan O'Sullivan
Allow tests to run in parallel.
r5384 pushing to http://localhost/
Vadim Gelfer
add tests for push over http.
r2481 searching for changes
Dirkjan Ochtman
tests: easier hg serve error diagnosis
r6167 % serve errors