diff options
author | Janak Ramakrishnan <janakr@google.com> | 2016-05-17 17:46:46 +0000 |
---|---|---|
committer | Kristina Chodorow <kchodorow@google.com> | 2016-05-17 21:58:01 +0000 |
commit | 8681777bd280103cd2000d6dfc8c81228b7b8dbf (patch) | |
tree | 6c4c55e6f98d0acdd136c6e8122644bd897cd3db /src/main/java | |
parent | 10ecf91e21e6522f0b483df729fb4af37f30c0f6 (diff) |
Batch package retrieval for rbuildfiles so that even if a single file has a large number of rbuildfile dependencies, we retrieve them in batches of at most BATCH_CALLBACK_SIZE.
--
MOS_MIGRATED_REVID=122540401
Diffstat (limited to 'src/main/java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/query2/SkyQueryEnvironment.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/query2/SkyQueryEnvironment.java b/src/main/java/com/google/devtools/build/lib/query2/SkyQueryEnvironment.java index c127f42570..9ab241f111 100644 --- a/src/main/java/com/google/devtools/build/lib/query2/SkyQueryEnvironment.java +++ b/src/main/java/com/google/devtools/build/lib/query2/SkyQueryEnvironment.java @@ -920,8 +920,10 @@ public class SkyQueryEnvironment extends AbstractBlazeQueryEnvironment<Target> { } } if (resultKeys.size() >= BATCH_CALLBACK_SIZE) { - callback.process( - getBuildFilesForPackageValues(graph.getSuccessfulValues(resultKeys).values())); + for (Iterable<SkyKey> batch : Iterables.partition(resultKeys, BATCH_CALLBACK_SIZE)) { + callback.process( + getBuildFilesForPackageValues(graph.getSuccessfulValues(batch).values())); + } resultKeys.clear(); } } |