markap14 commented on a change in pull request #5004:
URL: https://github.com/apache/nifi/pull/5004#discussion_r615852632
##########
File path: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java
##########
@@ -841,69 +905,17 @@ public void stop() throws IOException {
out.flush();
socket.shutdownOutput();
- final InputStream in = socket.getInputStream();
- int lastChar;
- final StringBuilder sb = new StringBuilder();
- while ((lastChar = in.read()) > -1) {
- sb.append((char) lastChar);
- }
- final String response = sb.toString().trim();
-
+ final String response = readResponse(socket.getInputStream());
Review comment:
Looks like we don't have IOUtils here, in the bootstrap. There is an
IOUtils available, but it's `sun.misc.IOUtils`, not apache commons or anything
like that.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
