fix:[2.5] avoid potential race conditions when updating the executor

[sparknack]

issue: #43030
pr: #45230

[sre-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sparknack
To complete the pull request process, please assign congqixia after the PR has been reviewed.
You can assign the PR to them by writing /assign @congqixia in a comment when ready.

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

Needs approval from an approver in each of these files:

• internal/core/OWNERS

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

[sre-ci-robot]

[INFO] PR Label Summary by Default
[FAILED] PR #45230 not merged

• Title: fix: avoid potential race conditions when updating the executor
• Status: OPEN
• Link: fix: avoid potential race conditions when updating the executor #45230

[WARNING] Milestone not set

• PR: fix:[2.5] avoid potential race conditions when updating the executor #45231
• Title: fix:[2.5] avoid potential race conditions when updating the executor
  Please set a milestone for better release tracking

You can set milestone by commenting:
/set-milestone
Example:
/set-milestone 2.5.0

Use /refresh-label to update related check and label manually

[sre-ci-robot]

[ci-v2-notice]
Notice: We are gradually rolling out the new ci-v2 system.

• Legacy CI jobs remain unaffected, you can just ignore ci-v2 if you don't want to run it.
• Additional "ci-v2/*" checkers will run for this PR to ensure the new ci-v2 system is working as expected.
• For tests that exist in both v1 and v2, passing in either system is considered PASS.

To rerun ci-v2 checks, comment with:

• /ci-rerun-code-check // for ci-v2/code-check
• /ci-rerun-build // for ci-v2/build
• /ci-rerun-ut-integration // for ci-v2/ut-integration
• /ci-rerun-ut-go // for ci-v2/ut-go
• /ci-rerun-ut-cpp // for ci-v2/ut-cpp
• /ci-rerun-ut // for all ci-v2/ut-integration, ci-v2/ut-go, ci-v2/ut-cpp
• /ci-rerun-e2e-arm // for ci-v2/e2e-arm

If you have any questions or requests, please contact @zhikunyao.

[mergify]

@sparknack cpu-e2e job failed, comment /run-cpu-e2e can trigger the job again.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.27%. Comparing base (3a7a08f) to head (301e8f4).
⚠️ Report is 36 commits behind head on 2.5.

❌ Your project check has failed because the head coverage (58.27%) is below the target coverage (77.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@             Coverage Diff             @@
##              2.5   #45231       +/-   ##
===========================================
- Coverage   82.10%   58.27%   -23.84%     
===========================================
  Files        1128     1469      +341     
  Lines      179181   238194    +59013     
===========================================
- Hits       147110   138797     -8313     
- Misses      26099    92366    +66267     
- Partials     5972     7031     +1059     

Components	Coverage Δ
Client	∅ <ø> (∅)
Core	84.64% <76.71%> (∅)
Go	47.87% <62.24%> (-34.51%)	⬇️

Files with missing lines	Coverage Δ
internal/core/src/storage/FileWriter.h	91.01% <100.00%> (ø)

... and 1323 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.