Github user MikeThomsen commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2594#discussion_r187743134
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteStreamCommand.java
---
@@ -382,10 +389,10 @@ public void onTrigger(ProcessContext context, final
ProcessSession session) thro
Map<String, String> attributes = new HashMap<>();
final StringBuilder strBldr = new StringBuilder();
- try {
- String line;
- while ((line = bufferedReader.readLine()) != null) {
- strBldr.append(line).append("\n");
+ try (final InputStream is = new FileInputStream(errorOut)) {
--- End diff --
The java-monitor.com link wouldn't load for me, so I couldn't see your
reference. So is this problem here with the BufferedReader? I'm not entirely
sure I follow the change here because either way you're pulling data from
stderr. This case just redirects it and you're reading it from a file.
---
