Remove unnecessary Invalid PBE rewrap
InvalidProtocolBufferException is already an IOException and does not
need to be rewrapped in one.
Closes
@@ -24,7 +24,6 @@ import com.google.common.base.Preconditions;
import com.google.devtools.build.lib.concurrent.ThreadSafety.ThreadSafe;
import com.google.devtools.build.lib.remote.util.TracingMetadataUtils;
import com.google.longrunning.Operation;
-import com.google.protobuf.InvalidProtocolBufferException;
import com.google.rpc.Status;
import io.grpc.CallCredentials;
import io.grpc.Status.Code;
@@ -73,7 +72,6 @@ class GrpcRemoteExecutor {
}
if (op.getDone()) {
Preconditions.checkState(op.getResultCase() != Operation.ResultCase.RESULT_NOT_SET);
- try {
ExecuteResponse resp = op.getResponse().unpack(ExecuteResponse.class);
if (resp.hasStatus()) {
handleStatus(resp.getStatus(), resp);
@@ -91,9 +89,6 @@ class GrpcRemoteExecutor {
"Unexpected result of remote execution: no output files.");
}
return resp;
- } catch (InvalidProtocolBufferException e) {
- throw new IOException(e);
- }
}
return null;
}
