codope commented on a change in pull request #5004:
URL: https://github.com/apache/hudi/pull/5004#discussion_r832444631
##########
File path: hudi-integ-test/src/test/java/org/apache/hudi/integ/ITTestBase.java
##########
@@ -236,8 +238,11 @@ private TestExecStartResultCallback
executeCommandInDocker(String containerName,
int exitCode =
dockerClient.inspectExecCmd(createCmdResponse.getId()).exec().getExitCode();
LOG.info("Exit code for command : " + exitCode);
if (exitCode != 0) {
- LOG.error("\n\n ###### Stdout #######\n" +
callback.getStdout().toString());
+ //LOG.error("\n\n ###### Stdout #######\n" +
callback.getStdout().toString());
Review comment:
Let's remove this condition, or maybe bring L243-L245 inside this
conditional?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
