Skip to content

Commit 386869a

Browse files
committed
GitMaintenanceScheduler: Remove PackfileMaintenanceStep
The PackfileMaintenanceStep introduced an issue on the perf machine during CI builds, so pull it out of the scheduler until we fullly understand what happened and have a fix prepared. While in the scheduler, I noticed that the non-prefetch maintenance was not running when we are not using a cache server. Fix that so those users can still benefit from the maintenance.
1 parent c2e6192 commit 386869a

1 file changed

Lines changed: 6 additions & 15 deletions

File tree

GVFS/GVFS.Common/Maintenance/GitMaintenanceScheduler.cs

Lines changed: 6 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ public class GitMaintenanceScheduler : IDisposable
1010
private readonly TimeSpan looseObjectsDueTime = TimeSpan.FromMinutes(5);
1111
private readonly TimeSpan looseObjectsPeriod = TimeSpan.FromHours(6);
1212

13-
private readonly TimeSpan packfileDueTime = TimeSpan.FromMinutes(30);
14-
private readonly TimeSpan packfilePeriod = TimeSpan.FromHours(12);
15-
1613
private readonly TimeSpan prefetchPeriod = TimeSpan.FromMinutes(15);
1714

1815
private List<Timer> stepTimers;
@@ -62,19 +59,13 @@ private void ScheduleRecurringSteps()
6259
state: null,
6360
dueTime: this.prefetchPeriod,
6461
period: this.prefetchPeriod));
65-
66-
this.stepTimers.Add(new Timer(
67-
(state) => this.queue.TryEnqueue(new LooseObjectsStep(this.context)),
68-
state: null,
69-
dueTime: this.looseObjectsDueTime,
70-
period: this.looseObjectsPeriod));
71-
72-
this.stepTimers.Add(new Timer(
73-
(state) => this.queue.TryEnqueue(new PackfileMaintenanceStep(this.context)),
74-
state: null,
75-
dueTime: this.packfileDueTime,
76-
period: this.packfilePeriod));
7762
}
63+
64+
this.stepTimers.Add(new Timer(
65+
(state) => this.queue.TryEnqueue(new LooseObjectsStep(this.context)),
66+
state: null,
67+
dueTime: this.looseObjectsDueTime,
68+
period: this.looseObjectsPeriod));
7869
}
7970
}
8071
}

0 commit comments

Comments
 (0)