bbende commented on issue #4154: NIFI-7271 Make command timeout configurable 
for ShellUserGroupProvider
URL: https://github.com/apache/nifi/pull/4154#issuecomment-602813340
 
 
   @alopresto thanks for reviewing! 
   
   During further testing, I found that we can't rely on calling 
`process.start()` and then calling `waitFor(...)` without every trying to read 
the output from the process. So I've added another commit that adds an Executor 
to the ShellRunner so after calling start() we submit a task to start reading 
from the input stream, then back in the main thread we call waitFor().
   
   I also made the change from long to int.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to