[HIVE-25001] Improvement for some debug-logging guards

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.0.0
• Fix Version/s: 4.0.0-alpha-1
• Component/s: Logging
• Labels: pull-request-available
• Assignee: Chen Zhi
• Reporter: Chen Zhi
• Created: 11/Apr/21 02:07
• Updated: 17/Nov/22 08:47
• Resolved: 14/Apr/21 09:01

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.

Issue Links

• relates to: HIVE-24913 LlapTaskScheduler Improvements
• links to: GitHub Pull Request #2165