Removing deprecated field total_input_file_count.
See API Revisions design doc:
The BuildInfo is now sent via RequestMetadata.
TESTED=we never used this field
@@ -173,7 +173,13 @@ class RemoteSpawnRunner implements SpawnRunner {
final ActionResult result;
try {
- result = executeRemotely(action, inputMap.size(), acceptCachedResult);
+ ExecuteRequest.Builder request =
+ ExecuteRequest.newBuilder()
+ .setInstanceName(options.remoteInstanceName)
+ .setAction(action)
+ .setSkipCacheLookup(!acceptCachedResult);
+ ExecuteResponse reply = remoteExecutor.executeRemotely(request.build());
+ result = reply.getResult();
} catch (IOException e) {
return execLocallyOrFail(spawn, policy, inputMap, actionKey, uploadLocalResults, e);
}
@@ -202,19 +208,6 @@ class RemoteSpawnRunner implements SpawnRunner {
.build();
}
- private ActionResult executeRemotely(Action action, int numInputFiles, boolean acceptCachedResult)
- throws IOException, InterruptedException {
- // TODO(olaola): set BuildInfo and input total bytes as well.
- ExecuteRequest.Builder request =
- ExecuteRequest.newBuilder()
- .setInstanceName(options.remoteInstanceName)
- .setAction(action)
- .setTotalInputFileCount(numInputFiles)
- .setSkipCacheLookup(!acceptCachedResult);
- ExecuteResponse reply = remoteExecutor.executeRemotely(request.build());
- return reply.getResult();
- }
-
private SpawnResult execLocallyOrFail(
Spawn spawn,
SpawnExecutionPolicy policy,
