diff options
author | janakr <janakr@google.com> | 2018-08-13 16:13:42 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-08-13 16:15:46 -0700 |
commit | 6ae05b419922193c4c253e51c9a5e483e4f947fa (patch) | |
tree | e06491f807689124cad73a2011af2db821e390e1 /src/test/java/com/google/devtools/build/lib/actions/MapBasedActionGraphTest.java | |
parent | ce170540ba5401e926a5433e6e35b1d22426e525 (diff) |
Order Skyframe evaluations in a priority queue, with all children of a given node having the same priority, later enqueueings having higher priority, re-enqueued nodes having highest priority, and new root nodes having lowest priority. Experimentally, this can save significant RAM (1.4G in some builds!) while not affecting speed.
Also do a semi-drive-by deleting ExecutorFactory parameter to AbstractQueueVisitor, since it was always AbstractQueueVisitor.EXECUTOR_FACTORY.
PiperOrigin-RevId: 208560889
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/actions/MapBasedActionGraphTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/actions/MapBasedActionGraphTest.java | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/actions/MapBasedActionGraphTest.java b/src/test/java/com/google/devtools/build/lib/actions/MapBasedActionGraphTest.java index 5de14d956c..5ec0d4a58f 100644 --- a/src/test/java/com/google/devtools/build/lib/actions/MapBasedActionGraphTest.java +++ b/src/test/java/com/google/devtools/build/lib/actions/MapBasedActionGraphTest.java @@ -89,7 +89,6 @@ public class MapBasedActionGraphTest { TimeUnit.SECONDS, /*failFastOnException=*/ true, "action-graph-test", - AbstractQueueVisitor.EXECUTOR_FACTORY, ErrorClassifier.DEFAULT); Path execRoot = fileSystem.getPath("/"); Path root = fileSystem.getPath("/root"); |