Format work requests according to ndjson spec

wolfd · danny-skydio · commit 91df9fb393d9 · 2021-11-02T16:05:12.000-07:00
JSON persistent workers now delimit requests with newlines
diff --git a/src/main/java/com/google/devtools/build/lib/worker/JsonWorkerProtocol.java b/src/main/java/com/google/devtools/build/lib/worker/JsonWorkerProtocol.java
@@ -48,7 +48,10 @@ final class JsonWorkerProtocol implements WorkerProtocolImpl {
 
   @Override
   public void putRequest(WorkRequest request) throws IOException {
+    // WorkRequests are serialized according to ndjson spec.
+    // https://github.com/ndjson/ndjson-spec
     jsonPrinter.appendTo(request, jsonWriter);
+    jsonWriter.append(System.lineSeparator());
     jsonWriter.flush();
   }
 

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,10 @@ final class JsonWorkerProtocol implements WorkerProtocolImpl {`
`48`	`48`
`49`	`49`	`@Override`
`50`	`50`	`public void putRequest(WorkRequest request) throws IOException {`
	`51`	`+ // WorkRequests are serialized according to ndjson spec.`
	`52`	`+ // https://github.com/ndjson/ndjson-spec`
`51`	`53`	`jsonPrinter.appendTo(request, jsonWriter);`
	`54`	`+ jsonWriter.append(System.lineSeparator());`
`52`	`55`	`jsonWriter.flush();`
`53`	`56`	`}`
`54`	`57`