nsivabalan commented on a change in pull request #1945:
URL: https://github.com/apache/hudi/pull/1945#discussion_r468265130



##########
File path: hudi-integ-test/src/test/java/org/apache/hudi/integ/ITTestBase.java
##########
@@ -103,22 +103,26 @@ private static String getHiveConsoleCommandFile(String 
commandFile, String addit
     if (additionalVar != null) {
       builder.append(" --hivevar " + additionalVar + " ");
     }
-    return builder.append(" -f ").append(commandFile).toString();
+    builder.append(" -f ").append(commandFile);
+    System.out.println("Hive command : " + builder.toString());

Review comment:
       this was useful in general while checking the Integ tests log output. If 
not for this, it wasn't easy to understand where the tests fails if tests hung 
or terminated half way if not for explicit exception. 




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to