ivankelly opened a new pull request #2249: Throw exception on non-zero command exit in integ tests URL: https://github.com/apache/incubator-pulsar/pull/2249 This patch replaces the boolean flag to ignore failure, and instead always throws an exception if a command exits with non-zero. This means that test code doesn't need to check the exit code for each invokation. For non-zero exits, the ContainerExecResult is carried as part of the exception, so the test can also assert on the contents of stdout, stderr and the exit code. ### Motivation Explain here the context, and why you're making that change. What is the problem you're trying to solve. ### Modifications Describe the modifications you've done. ### Result After your change, what will change.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
