[jira] [Commented] (HIVE-25001) Improvement for some debug-logging guards

["Panagiotis Garefalakis (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HIVE-25001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17320821#comment-17320821 ] 

Panagiotis Garefalakis commented on HIVE-25001:
-----------------------------------------------

This is now resolved via https://github.com/apache/hive/pull/2165
Thanks for the PR [~coder_chenzhi] !

> Improvement for some debug-logging guards
> -----------------------------------------
>
>                 Key: HIVE-25001
>                 URL: https://issues.apache.org/jira/browse/HIVE-25001
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Chen Zhi
>            Assignee: Chen Zhi
>            Priority: Minor
>              Labels: pull-request-available
>         Attachments: HIVE-25001.patch
>
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> Moving some potential expensive statements (e.g. `Arrays.toString`) into the adjacent logging guards, since these statements are merely used by the following logging calls.
> Note that the fix in `LlapTaskSchedulerService.java` is different with other cases in this patch, because the variable `requestedHostsDebugStr` is used by several debug-logging calls, and we have to declare and assign the variable before these debug-logging calls.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)