tiflow icon indicating copy to clipboard operation
tiflow copied to clipboard

redo(ticdc): enable pprof and set memory limit for redo applier (#10904)

Open ti-chi-bot opened this issue 10 months ago • 2 comments

This is an automated cherry-pick of #10904

What problem does this PR solve?

Issue Number: close #10900

What is changed and how it works?

  1. Set memory limit for redo applier.
  2. Enable pprof for redo applier to simplify troubleshooting.

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Questions

Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?

Release note

Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

ti-chi-bot avatar Apr 27 '24 14:04 ti-chi-bot

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign charlescheung96 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

ti-chi-bot[bot] avatar Apr 27 '24 14:04 ti-chi-bot[bot]

@ti-chi-bot: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
jenkins-ticdc/verify de21c330392a753d14205eeb35ee24b6967d6b00 link true /test verify
pull-cdc-integration-mysql-test de21c330392a753d14205eeb35ee24b6967d6b00 link true /test cdc-integration-mysql-test
pull-dm-integration-test de21c330392a753d14205eeb35ee24b6967d6b00 link true /test dm-integration-test
pull-cdc-integration-kafka-test de21c330392a753d14205eeb35ee24b6967d6b00 link true /test cdc-integration-kafka-test
pull-dm-compatibility-test de21c330392a753d14205eeb35ee24b6967d6b00 link true /test dm-compatibility-test

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

ti-chi-bot[bot] avatar Apr 27 '24 14:04 ti-chi-bot[bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CharlesCheung96

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • ~~OWNERS~~ [CharlesCheung96]

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

ti-chi-bot[bot] avatar Jun 06 '24 09:06 ti-chi-bot[bot]

/retest

CharlesCheung96 avatar Jun 07 '24 07:06 CharlesCheung96