Closed windtalker closed 6 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
90991fb
) 55.53% compared to head (58fac00
) 55.46%.:exclamation: Current head 58fac00 differs from pull request most recent head fd21937. Consider uploading reports for the commit fd21937 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@JaySon-Huang: adding LGTM is restricted to approvers and reviewers in OWNERS files.
@breezewish PTAL
/lgtm
/run-all-tests
/hold
/lgtm
Timeline:
2024-01-04 03:00:06.179462439 +0000 UTC m=+2312297.216689365
: :ballot_box_with_check: agreed by kaaaaaaang.2024-01-04 07:13:48.495691742 +0000 UTC m=+2327519.532918657
: :heavy_multiplication_x::repeat: reset by windtalker.2024-01-04 07:34:09.899049525 +0000 UTC m=+2328740.936276452
: :ballot_box_with_check: agreed by kaaaaaaang./approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kaaaaaaang
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What problem does this PR solve?
delete logger.level and profiles.default.max_memory_usage in tiflash default configs since there is no need to set them explicitly in the config file.
What is changed and how it works?
Check List
Tests
Code changes
Side effects
Related changes
Release notes: