shuyouZZ opened a new pull request, #5848:
URL: https://github.com/apache/hive/pull/5848

   ### What changes were proposed in this pull request?
   Add a space to debug log in endFunction.
   
   ### Why are the changes needed?
   The debug log in endFunction is missing a space, which leads to incorrect 
format. We should keep the correct format.
   
   ```
   private void endFunction(String function, MetaStoreEndFunctionContext 
context) {
       com.codahale.metrics.Timer.Context timerContext = 
HMSHandlerContext.getTimerContexts().remove(function);
       if (timerContext != null) {
         long timeTaken = timerContext.stop();
         LOG.debug((getThreadLocalIpAddress() == null ? "" : "source:" + 
getThreadLocalIpAddress() + " ") +
             function + "time taken(ns): " + timeTaken);
       }
      ...
     }
   ```
   The debug log shows before:
   ```
   [DEBUG] 2025-06-06 18:05:09.398 [main] HMSHandler - get_databasetime 
taken(ns): 45370281
   ```
   After this PR:
   ```
   [DEBUG] 2025-06-06 18:16:15.551 [main] HMSHandler - get_database time 
taken(ns): 45370281
   ```
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   Just change the debug log format
   


-- 
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

Reply via email to