ramesh0201 commented on code in PR #4755:
URL: https://github.com/apache/hive/pull/4755#discussion_r1442987243
##########
common/src/java/org/apache/hadoop/hive/ql/log/PerfLogger.java:
##########
@@ -207,6 +241,14 @@ public Map<String, Long> getStartTimes() {
return ImmutableMap.copyOf(startTimes);
}
+ public Set<String> getQuerySteps() {
+ return querySteps;
+ }
+
+ public Map<String, Long> getDurations() {
+ return durations;
+ }
+
Review Comment:
Done I will do this change
##########
ql/src/java/org/apache/hadoop/hive/ql/Compiler.java:
##########
@@ -504,6 +506,15 @@ private void cleanUp(Throwable compileException, boolean
parsed, boolean deferCl
driverState.compilationFinishedWithLocking(compileException != null);
LOG.info("Completed compiling command(queryId={}); Time taken: {}
seconds", driverContext.getQueryId(),
duration);
+ if (HiveConf.getBoolVar(driverContext.getConf(),
HiveConf.ConfVars.HIVE_COMPILE_SUMMARY)) {
+ LogHelper console = SessionState.getConsole();
+ console.printInfo("Query Compilation Summary");
+ console.printInfo(SEPARATOR);
+ for (String qs : perfLogger.getQuerySteps()) {
+ console.printInfo(String.format("%-99s %9s", qs,
perfLogger.getDurations().get(qs) + " ms"));
+ }
+ console.printInfo(SEPARATOR);
+ }
Review Comment:
Done, I will do this change
--
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: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
