[ 
https://issues.apache.org/jira/browse/HIVE-25001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chen Zhi updated HIVE-25001:
----------------------------
    Description: 
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.

  was: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.


> 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
>            Priority: Minor
>         Attachments: HIVE-25001.patch
>
>
> 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)

Reply via email to