diff options
author | janakr <janakr@google.com> | 2017-03-28 22:39:35 +0000 |
---|---|---|
committer | Philipp Wollermann <philwo@google.com> | 2017-03-29 14:22:26 +0200 |
commit | db4dec23008069a4c5ceccdb686e515808c921ba (patch) | |
tree | acec73f2e74a6f593c5f0e7dd5df22450654a15f /src/test/java/com | |
parent | 0e866a8b732e31df1a4ddfd13c4870d4721de70b (diff) |
Clear AspectValues when discarding analysis cache, along with ConfiguredTargetValues. Also clear transitive packages for both, even for top-level targets.
This is not expected to save significant memory, but is expected to reduce the number of references to Packages, allowing them to be dropped more easily when discarding analysis cache and running in batch mode.
PiperOrigin-RevId: 151508877
Diffstat (limited to 'src/test/java/com')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/analysis/AnalysisCachingTest.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/AnalysisCachingTest.java b/src/test/java/com/google/devtools/build/lib/analysis/AnalysisCachingTest.java index 3093d8de4c..2a86b99514 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/AnalysisCachingTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/AnalysisCachingTest.java @@ -22,10 +22,12 @@ import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableSet; import com.google.devtools.build.lib.actions.Action; import com.google.devtools.build.lib.analysis.util.AnalysisCachingTestBase; import com.google.devtools.build.lib.events.Event; import com.google.devtools.build.lib.rules.java.JavaSourceJarsProvider; +import com.google.devtools.build.lib.skyframe.AspectValue; import com.google.devtools.build.lib.testutil.Suite; import com.google.devtools.build.lib.testutil.TestSpec; import java.util.Set; @@ -200,7 +202,8 @@ public class AnalysisCachingTest extends AnalysisCachingTestBase { "//conflict:x", "//conflict:_objs/x/conflict/foo.pic.o"); // We want to force a "dropConfiguredTargetsNow" operation, which won't inform the // invalidation receiver about the dropped configured targets. - getView().clearAnalysisCache(ImmutableList.<ConfiguredTarget>of()); + skyframeExecutor.clearAnalysisCache( + ImmutableList.<ConfiguredTarget>of(), ImmutableSet.<AspectValue>of()); assertContainsEvent("file 'conflict/_objs/x/conflict/foo.pic.o' " + CONFLICT_MSG); eventCollector.clear(); scratch.overwriteFile("conflict/BUILD", |