Skip to content

[Improvement][Common] Remove the code logger.isDebugEnabled() #5918

@zhuangchong

Description

@zhuangchong

Describe the question

  1. When the log is printed using string splicing, when the program judges the level of the print log, it will splice the log information of the string first. If the sentence does not print the string in the end, then this spliced string The process is a meaningless operation, which will increase the loss of system performance. At this time, it is recommended to use logger.isDebugEnabled()

  2. Use the {} placeholder to print the log, there is no need to prejudge the level of the log, the bottom layer has already implemented this step.

The current system uses {} placeholders to print logs, so remove the code logger.isDebugEnabled().

What are the current deficiencies and the benefits of improvement

  • Repeated calls to determine whether it is debug log level

Which version of DolphinScheduler:
-[dev]

Metadata

Metadata

Assignees

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions