mirror of
https://github.com/facebook/sapling.git
synced 2024-10-10 16:57:49 +03:00
f1ecf43a83
Summary: Parallel callout to scmquery through conduit This implements the special case of log on a single directory, preserving follow behavior. To do this, we backtrackfrom the current head to find all draft revisions along the path, then find the common public ancestor of those. Once we find that, we can begin paging in results from the scmquery service. Pretty much a working diff at this point. Limits and boundary conditions have not been fully tested. Every once in a while I run into a bum query, which I suspect to be either a bad proxy or a service router failure; still debugging that. Could also be an issue with conduit. Other than that, things seem to work. Test Plan: Testing log with fastest setting (no revsets), revsets (using -M), and with extension disabled (--sparse): [zamsden@devbig192.prn1 ~/local/fbcode] time hg log tao --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 100 -M > a real 0m1.895s user 0m0.003s sys 0m0.004s [zamsden@devbig192.prn1 ~/local/fbcode] time hg log tao --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 100 > b real 0m1.308s user 0m0.005s sys 0m0.001s [zamsden@devbig192.prn1 ~/local/fbcode] diff a b [zamsden@devbig192.prn1 ~/local/fbcode] time hg log tao --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 100 --sparse > c real 0m7.320s user 0m0.004s sys 0m0.003s [zamsden@devbig192.prn1 ~/local/fbcode] diff a c Testing --user option: [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/dragon --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 5 -u dmitri > a real 0m2.765s user 0m0.002s sys 0m0.004s [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/dragon --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 5 -u dmitri --sparse > b real 0m23.247s user 0m0.005s sys 0m0.001s [zamsden@devbig192.prn1 ~/local/fbsource] diff a b [zamsden@devbig192.prn1 ~/local/fbsource] Testing same output enabled / disabled for -X option: [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/lithium/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 20 -X '**TARGETS' > a real 0m1.292s user 0m0.002s sys 0m0.003s [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/lithium/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 20 -X '**TARGETS' --sparse > b real 0m2.697s user 0m0.002s sys 0m0.004s [zamsden@devbig192.prn1 ~/local/fbsource] diff a b [zamsden@devbig192.prn1 ~/local/fbsource] Testing -k option: [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/lithium/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -k 'e' -X '**TARGETS' -l 10 > a real 0m1.174s user 0m0.003s sys 0m0.003s [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/lithium/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -k 'e' -X '**TARGETS' -l 10 --sparse > b real 0m1.259s user 0m0.004s sys 0m0.002s [zamsden@devbig192.prn1 ~/local/fbsource] diff a b [zamsden@devbig192.prn1 ~/local/fbsource] Testing -I option: [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/scm/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 20 -I '**.py' > a real 0m1.473s user 0m0.005s sys 0m0.003s [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/scm/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 20 -I '**.py' --sparse > b real 0m2.911s user 0m0.003s sys 0m0.002s [zamsden@devbig192.prn1 ~/local/fbsource] diff a b [zamsden@devbig192.prn1 ~/local/fbsource] Testing multiple directory output in all three modes - revset, fast filtered, and forcing original fallback with --sparse [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/hphp/hhvm fbcode/hphp/runtime/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 100 -M > a real 0m2.892s user 0m0.003s sys 0m0.006s [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/hphp/hhvm fbcode/hphp/runtime/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 100 > b real 0m2.575s user 0m0.697s sys 0m0.077s [zamsden@devbig192.prn1 ~/local/fbsource] time hg log fbcode/hphp/hhvm fbcode/hphp/runtime/ --config extensions.fbconduit=~/local/fb-hgext/hgext3rd/fbconduit.py --config fbconduit.host=our.zamsden.devbig192.prn1.facebook.com --config extensions.fastlog=~/local/fb-hgext/hgext3rd/fastlog.py --pager=off -l 100 --sparse > c real 0m7.339s user 0m0.003s sys 0m0.004s [zamsden@devbig192.prn1 ~/local/fbsource] diff a b [zamsden@devbig192.prn1 ~/local/fbsource] diff a c [zamsden@devbig192.prn1 ~/local/fbsource] Reviewers: rmcelroy, #scmquery, #mercurial, ttung, lcharignon, durham, stash Reviewed By: stash Subscribers: cdykes, lcharignon, quark, stash, mjpieters, jeroenv Differential Revision: https://phabricator.intern.facebook.com/D3634075 Tasks: 12341014 Signature: t1:3634075:1471039212:0989839636847a8e5da6a0ef9150035fcf5bb797
126 lines
2.1 KiB
Perl
126 lines
2.1 KiB
Perl
$ extpath=`dirname $TESTDIR`
|
|
$ cat >> $HGRCPATH << EOF
|
|
> [extensions]
|
|
> tweakdefaults=$extpath/hgext3rd/tweakdefaults.py
|
|
> fastlog=$extpath/hgext3rd/fastlog.py
|
|
> fbconduit=$extpath/hgext3rd/fbconduit.py
|
|
> [fbconduit]
|
|
> host=our.intern.facebook.com
|
|
> protocol=http
|
|
> reponame=fbsource
|
|
> path=/intern/conduit/
|
|
> [fastlog]
|
|
> enabled=True
|
|
> EOF
|
|
|
|
Log on empty repo
|
|
|
|
$ hg init repo
|
|
$ cd repo
|
|
$ mkdir dir
|
|
$ hg log dir
|
|
$ hg log dir -M
|
|
|
|
Create a directory and test some log commands
|
|
|
|
$ touch dir/a
|
|
$ hg commit -Aqm a
|
|
$ hg log dir -T '{rev} {desc}\n'
|
|
0 a
|
|
$ hg log dir -T '{rev} {desc}\n' -M
|
|
0 a
|
|
$ hg log dir -T '{rev} {desc}\n' --all
|
|
0 a
|
|
$ echo x >> dir/a
|
|
$ hg commit -Aqm a2
|
|
$ hg up -q 0
|
|
$ touch dir/b
|
|
$ hg commit -Aqm b
|
|
$ hg log dir -T '{rev} {desc}\n'
|
|
2 b
|
|
0 a
|
|
$ hg log dir -T '{rev} {desc}\n' --all
|
|
2 b
|
|
1 a2
|
|
0 a
|
|
$ hg log dir -r 'draft()' -T '{rev} {desc}\n'
|
|
0 a
|
|
1 a2
|
|
2 b
|
|
|
|
Graphlog still works
|
|
|
|
$ hg log dir -G -T '{rev} {desc}\n'
|
|
@ 2 b
|
|
|
|
|
o 0 a
|
|
|
|
|
|
$ hg log dir -G -T '{rev} {desc}\n' --all
|
|
@ 2 b
|
|
|
|
|
| o 1 a2
|
|
|/
|
|
o 0 a
|
|
|
|
|
|
Create a merge
|
|
|
|
$ hg merge --config tweakdefaults.allowmerge=True
|
|
1 files updated, 0 files merged, 0 files removed, 0 files unresolved
|
|
(branch merge, don't forget to commit)
|
|
$ hg log -T '{rev} {desc}\n'
|
|
2 b
|
|
0 a
|
|
$ hg commit -Aqm merge
|
|
$ hg log -T '{rev} {desc}\n'
|
|
3 merge
|
|
2 b
|
|
1 a2
|
|
0 a
|
|
$ hg log dir -T '{rev} {desc}\n'
|
|
2 b
|
|
1 a2
|
|
0 a
|
|
$ hg log dir -T '{rev} {desc}\n' -M
|
|
2 b
|
|
1 a2
|
|
0 a
|
|
|
|
Test keywords
|
|
|
|
$ hg log dir -k 2 -T '{rev} {desc}\n'
|
|
1 a2
|
|
|
|
Test pruning
|
|
|
|
$ hg log dir -P 1 -T '{rev} {desc}\n'
|
|
2 b
|
|
$ hg log dir -P 2 -T '{rev} {desc}\n'
|
|
1 a2
|
|
|
|
Create a public ancestor
|
|
$ hg up 0 -q
|
|
$ hg phase --public .
|
|
$ hg log dir -T '{rev} {desc}\n'
|
|
0 a
|
|
$ hg up 3 -q
|
|
$ hg log dir -T '{rev} {desc}\n'
|
|
2 b
|
|
1 a2
|
|
0 a
|
|
|
|
Test include / exclude
|
|
$ hg log dir -I 'dir/a' -T '{rev} {desc}\n'
|
|
1 a2
|
|
0 a
|
|
$ hg log dir -X 'dir/a' -T '{rev} {desc}\n'
|
|
2 b
|
|
|
|
Log on non-existent directory
|
|
|
|
$ hg log dir2
|
|
abort: cannot follow file not in parent revision: "dir2"
|
|
[255]
|
|
|