ramesh0201 commented on code in PR #4755: URL: https://github.com/apache/hive/pull/4755#discussion_r1442985426
########## common/src/java/org/apache/hadoop/hive/ql/log/PerfLogger.java: ########## @@ -129,9 +133,31 @@ public static void setPerfLogger(PerfLogger resetPerfLogger) { * @param method method or ID that identifies this perf log element. */ public void perfLogBegin(String callerName, String method) { + perfLogBegin(callerName, method, null); + } + + /** + * Call this function when you start to measure time spent by a piece of code. + * @param callerName the logging object to be used. + * @param method method or ID that identifies this perf log element. + * @param additionalInfo particular step within the method that is being measured + */ + public void perfLogBegin(String callerName, String method, String additionalInfo) { long startTime = System.currentTimeMillis(); - startTimes.put(method, Long.valueOf(startTime)); - LOG.debug("<PERFLOG method={} from={}>", method, callerName); + String key = method; + if (additionalInfo != null) { + key = key + " - " + additionalInfo; Review Comment: Makes sense, 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