Skip to content

Kill RDB child process on CLUSTER RESET command#1926

Merged
hpatro merged 1 commit intovalkey-io:unstablefrom
hpatro:cluster_reset_kill_rdb_process
Apr 7, 2025
Merged

Kill RDB child process on CLUSTER RESET command#1926
hpatro merged 1 commit intovalkey-io:unstablefrom
hpatro:cluster_reset_kill_rdb_process

Conversation

@hpatro
Copy link
Contributor

@hpatro hpatro commented Apr 7, 2025

Fixes: #1925

@hpatro hpatro assigned PingXie and unassigned PingXie Apr 7, 2025
@hpatro hpatro requested a review from PingXie April 7, 2025 20:34
@codecov
Copy link

codecov bot commented Apr 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.05%. Comparing base (add716b) to head (643ef95).
Report is 5 commits behind head on unstable.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable    #1926      +/-   ##
============================================
+ Coverage     71.02%   71.05%   +0.02%     
============================================
  Files           123      123              
  Lines         65683    65683              
============================================
+ Hits          46653    46671      +18     
+ Misses        19030    19012      -18     
Files with missing lines Coverage Δ
src/cluster_legacy.c 85.97% <100.00%> (+0.23%) ⬆️

... and 12 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@PingXie PingXie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hpatro hpatro merged commit 772bd8b into valkey-io:unstable Apr 7, 2025
51 checks passed
enjoy-binbin added a commit to enjoy-binbin/valkey that referenced this pull request Apr 8, 2025
We use lazyfree-lazy-user-flush in valkey-io#1190 but in valkey-io#1926 it was changed to
use replica-lazy-flush wrongly.

Signed-off-by: Binbin <[email protected]>
murphyjacob4 pushed a commit to enjoy-binbin/valkey that referenced this pull request Apr 13, 2025
hwware pushed a commit that referenced this pull request Apr 17, 2025
…1931)

We use lazyfree-lazy-user-flush in #1190 but in #1926 it was changed to
use replica-lazy-flush wrongly.

Signed-off-by: Binbin <[email protected]>
nitaicaro pushed a commit to nitaicaro/valkey that referenced this pull request Apr 22, 2025
…alkey-io#1931)

We use lazyfree-lazy-user-flush in valkey-io#1190 but in valkey-io#1926 it was changed to
use replica-lazy-flush wrongly.

Signed-off-by: Binbin <[email protected]>
Signed-off-by: Nitai Caro <[email protected]>
nitaicaro pushed a commit to nitaicaro/valkey that referenced this pull request Apr 22, 2025
…alkey-io#1931)

We use lazyfree-lazy-user-flush in valkey-io#1190 but in valkey-io#1926 it was changed to
use replica-lazy-flush wrongly.

Signed-off-by: Binbin <[email protected]>
nitaicaro pushed a commit to nitaicaro/valkey that referenced this pull request Apr 22, 2025
…alkey-io#1931)

We use lazyfree-lazy-user-flush in valkey-io#1190 but in valkey-io#1926 it was changed to
use replica-lazy-flush wrongly.

Signed-off-by: Binbin <[email protected]>
hwware pushed a commit to wuranxx/valkey that referenced this pull request Apr 24, 2025
…alkey-io#1931)

We use lazyfree-lazy-user-flush in valkey-io#1190 but in valkey-io#1926 it was changed to
use replica-lazy-flush wrongly.

Signed-off-by: Binbin <[email protected]>
Signed-off-by: hwware <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Kill RDB child process on CLUSTER RESET

2 participants