From 06cd576bb9b73eb177728fead06de8c3fac8d7a3 Mon Sep 17 00:00:00 2001 From: "Eric Wong (Contractor, The Linux Foundation)" Date: Sat, 7 Apr 2018 03:41:49 +0000 Subject: over: remove forked subprocess Since the overview stuff is a synchronization point anyways, move it into the main V2Writable process and allow us to drop a bunch of code. This is another step towards making Xapian optional for v2. In other words, the fan-out point is moved and the Xapian partitions no longer need to synchronize against each other: Before: /-------->\ /---------->\ v2writable -->+----parts----> over \---------->/ \-------->/ After: /----------> /-----------> v2writable --> over-->+----parts---> \-----------> \----------> Since the overview/threading logic needs to run on the same core that feeds git-fast-import, it's slower for small repos but is not noticeable in large imports where I/O wait in the partitions dominates. --- MANIFEST | 1 - 1 file changed, 1 deletion(-) (limited to 'MANIFEST') diff --git a/MANIFEST b/MANIFEST index 82cc67d3..58b3634b 100644 --- a/MANIFEST +++ b/MANIFEST @@ -84,7 +84,6 @@ lib/PublicInbox/NNTPD.pm lib/PublicInbox/NewsWWW.pm lib/PublicInbox/Over.pm lib/PublicInbox/OverIdx.pm -lib/PublicInbox/OverIdxFork.pm lib/PublicInbox/ParentPipe.pm lib/PublicInbox/ProcessPipe.pm lib/PublicInbox/Qspawn.pm -- cgit v1.2.3-24-ge0c7