[BugFix] Fix bRPC stub cache clean timer leak (backport #75973) by mergify[bot] · Pull Request #76361 · StarRocks/starrocks

mergify · 2026-07-14T12:06:39Z

Why I'm doing:

there is mem leak caused by bRPC stub cache clean timer leak due to timer is unscheduled but not removed from mem.

std::shared_ptr<PInternalService_RecoverableStub> BrpcStubCache::get_stub(const butil::EndPoint& endpoint) {
    ...
    if (_timer->unschedule((*stub_pool)->_cleanup_task.get()) != TIMER_TASK_RUNNING) {
        timespec tm = butil::seconds_from_now(config::brpc_stub_expire_s);
        auto status = _timer->schedule((*stub_pool)->_cleanup_task.get(), tm);
        if (!status.ok()) {
            LOG(WARNING) << "Failed to schedule brpc cleanup task: " << endpoint;
        }
    }
}

every get_stub() will do _timer->unschedule() and _timer->schedule().
unlike what we expect, _timer->unschedule() will not remove internal objects, thus leak happens

What I'm doing:

extend timer deadline if the stub is still in use instead of unschedule()

Fixes #issue

What type of PR is this:

Does this PR entail a change in behavior?

Yes, this PR will result in a change in behavior.
No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

Interface/UI changes: syntax, type conversion, expression evaluation, display information
Parameter changes: default values, similar parameters but with different default values
Policy changes: use new policy to replace old one, functionality automatically enabled
Feature removed
Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

I have added test cases for my bug fix or my new feature
This pr needs user documentation (for new or modified features or behaviors)
- I have added documentation for my new feature or new function
- This pr needs auto generate documentation
This is a backport pr

Bugfix cherry-pick branch check:

I have checked the version labels which the pr will be auto-backported to the target branch
- 4.1
- 4.0
- 3.5

This is an automatic backport of pull request #75973 done by [Mergify](https://mergify.com).

Signed-off-by: kaijian.ding <kaijian.ding@gmail.com> Co-authored-by: wanpengfei-git <wanpengfei91@163.com> (cherry picked from commit 2bbca67) # Conflicts: # be/src/util/brpc_stub_cache.cpp # be/src/util/brpc_stub_cache.h # be/test/util/brpc_stub_cache_test.cpp

mergify · 2026-07-14T12:06:41Z

Cherry-pick of 2bbca67 has failed:

On branch mergify/bp/branch-4.0/pr-75973
Your branch is up to date with 'origin/branch-4.0'.

You are currently cherry-picking commit 2bbca67281.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   be/src/util/brpc_stub_cache.cpp
	both modified:   be/src/util/brpc_stub_cache.h
	both modified:   be/test/util/brpc_stub_cache_test.cpp

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

mergify · 2026-07-14T12:07:28Z

@mergify[bot]: Backport conflict, please reslove the conflict and resubmit the pr

satanson · 2026-07-21T12:09:58Z

Cherry-pick of 2bbca67 has failed:

On branch mergify/bp/branch-4.0/pr-75973
Your branch is up to date with 'origin/branch-4.0'.

You are currently cherry-picking commit 2bbca67281.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   be/src/util/brpc_stub_cache.cpp
	both modified:   be/src/util/brpc_stub_cache.h
	both modified:   be/test/util/brpc_stub_cache_test.cpp

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@copilot fix backport conflicts

satanson · 2026-07-22T02:40:18Z

Cherry-pick of 2bbca67 has failed:

On branch mergify/bp/branch-4.0/pr-75973
Your branch is up to date with 'origin/branch-4.0'.

You are currently cherry-picking commit 2bbca67281.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   be/src/util/brpc_stub_cache.cpp
	both modified:   be/src/util/brpc_stub_cache.h
	both modified:   be/test/util/brpc_stub_cache_test.cpp

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@copilot please fix backport conflicts

Copilot · 2026-07-22T02:47:24Z

...

Addressed in commit 6ed30fa. The backport conflicts are resolved and the branch now contains the conflict-free implementation and test updates.

Cherry-pick of 2bbca67 has failed:

...

Fixed in commit 6ed30fa. I resolved the backport conflicts in be/src/util/brpc_stub_cache.{h,cpp} and be/test/util/brpc_stub_cache_test.cpp with the branch-4.0-compatible timer-leak fix.

…test

mergify Bot added the conflicts label Jul 14, 2026

github-actions Bot added the automerge label Jul 14, 2026

wanpengfei-git enabled auto-merge (squash) July 14, 2026 12:07

mergify Bot mentioned this pull request Jul 14, 2026

[BugFix] Fix bRPC stub cache clean timer leak #75973

Merged

23 tasks

mergify Bot closed this Jul 14, 2026

auto-merge was automatically disabled July 14, 2026 12:07
Pull request was closed

satanson reopened this Jul 21, 2026

wanpengfei-git enabled auto-merge (squash) July 21, 2026 12:10

satanson removed the conflicts label Jul 22, 2026

Copilot started work on behalf of satanson July 22, 2026 02:40 View session

Fix backport conflicts for brpc stub cache timer leak

6ed30fa

auto-merge was automatically disabled July 22, 2026 02:44
Head branch was pushed to by a user without write access

Copilot finished work on behalf of satanson July 22, 2026 02:47

Copilot AI requested a review from satanson July 22, 2026 02:47

wanpengfei-git enabled auto-merge (squash) July 22, 2026 03:11

style: auto-fix clang-format issues

15afe15

Copilot started work on behalf of satanson July 22, 2026 08:38 View session

Fix BE UT: add missing TIMER_TASK_RUNNING constant in pipeline_timer_…

e222926

…test

auto-merge was automatically disabled July 22, 2026 08:40
Head branch was pushed to by a user without write access

Copilot finished work on behalf of satanson July 22, 2026 08:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[BugFix] Fix bRPC stub cache clean timer leak (backport #75973)#76361

[BugFix] Fix bRPC stub cache clean timer leak (backport #75973)#76361
mergify[bot] wants to merge 4 commits into
branch-4.0from
mergify/bp/branch-4.0/pr-75973

mergify Bot commented Jul 14, 2026 •

edited by wanpengfei-git

Loading

Uh oh!

mergify Bot commented Jul 14, 2026

Uh oh!

mergify Bot commented Jul 14, 2026

Uh oh!

satanson commented Jul 21, 2026

Uh oh!

satanson commented Jul 22, 2026

Uh oh!

Copilot AI commented Jul 22, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

mergify Bot commented Jul 14, 2026 • edited by wanpengfei-git Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why I'm doing:

What I'm doing:

What type of PR is this:

Checklist:

Bugfix cherry-pick branch check:

Uh oh!

mergify Bot commented Jul 14, 2026

Uh oh!

mergify Bot commented Jul 14, 2026

Uh oh!

satanson commented Jul 21, 2026

Uh oh!

satanson commented Jul 22, 2026

Uh oh!

Copilot AI commented Jul 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

mergify Bot commented Jul 14, 2026 •

edited by wanpengfei-git

Loading

Copilot AI commented Jul 22, 2026 •

edited

Loading