diff options
| author |  philwo <philwo@google.com> | 2018-03-22 04:25:38 -0700 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-03-22 04:27:17 -0700 |
| commit | f10b56e8aba61618fb2f3ab3ba16e5a06220a51a (patch) | |
| tree | de237b0b784ffd9c5605a997aff6d00a60f51d0b /src/main/java/com/google/devtools/build/lib/worker | |
| parent | 72d51094d3bb5c835650abdb362a9bd4d1a2fef6 (diff) | |
Improve error message when a persistent worker sends an unparseable response.
Fixes #4897.
PiperOrigin-RevId: 190045930
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/worker')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java b/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java index f720d8cc12..1be61e1f77 100644 --- a/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java +++ b/src/main/java/com/google/devtools/build/lib/worker/WorkerSpawnRunner.java @@ -314,7 +314,10 @@ final class WorkerSpawnRunner implements SpawnRunner { recordingStream.readRemaining(); throw new UserExecException( ErrorMessage.builder() - .message("Worker process returned an unparseable WorkResponse:") + .message( + "Worker process returned an unparseable WorkResponse!\n\n" + + "Did you try to print something to stdout? Workers aren't allowed to do " + + "this, as it breaks the protocol between Bazel and the worker process.") .logText(recordingStream.getRecordedDataAsString()) .exception(e) .build() |