Skip to content

remove unneeded include of expire.h#3117

Merged
ranshid merged 1 commit into
valkey-io:unstablefrom
ranshid:fix-32bit-alignment-issue
Jan 27, 2026
Merged

remove unneeded include of expire.h#3117
ranshid merged 1 commit into
valkey-io:unstablefrom
ranshid:fix-32bit-alignment-issue

Conversation

@ranshid

@ranshid ranshid commented Jan 27, 2026

Copy link
Copy Markdown
Member

This caused some unaligned load of server values on 32bit compilations.

Example: https://github.com/valkey-io/valkey/actions/runs/21352969942/job/61491923381?pr=3111#step:6:3064

Some insights here: #3111 (comment)

Signed-off-by: Ran Shidlansik <ranshid@amazon.com>
@ranshid ranshid requested a review from zuiderkwast January 27, 2026 13:08
@ranshid ranshid moved this to To be backported in Valkey 9.0 Jan 27, 2026
@codecov

codecov Bot commented Jan 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.81%. Comparing base (4ad7628) to head (cdb825c).
⚠️ Report is 1 commits behind head on unstable.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable    #3117      +/-   ##
============================================
+ Coverage     74.75%   74.81%   +0.05%     
============================================
  Files           129      129              
  Lines         71145    71145              
============================================
+ Hits          53184    53225      +41     
+ Misses        17961    17920      -41     
Files with missing lines Coverage Δ
src/t_hash.c 95.38% <ø> (ø)

... and 19 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.

@ranshid ranshid merged commit 928e912 into valkey-io:unstable Jan 27, 2026
25 checks passed
ranshid added a commit to ranshid/valkey that referenced this pull request Jan 27, 2026
This caused some unaligned load of server values on 32bit compilations.

Example:
https://github.com/valkey-io/valkey/actions/runs/21352969942/job/61491923381?pr=3111#step:6:3064

Some insights here:
valkey-io#3111 (comment)

Signed-off-by: Ran Shidlansik <ranshid@amazon.com>
@zuiderkwast zuiderkwast moved this from To be backported to 9.0.2 WIP in Valkey 9.0 Jan 28, 2026
ranshid added a commit to ranshid/valkey that referenced this pull request Jan 28, 2026
This caused some unaligned load of server values on 32bit compilations.

Example:
https://github.com/valkey-io/valkey/actions/runs/21352969942/job/61491923381?pr=3111#step:6:3064

Some insights here:
valkey-io#3111 (comment)

Signed-off-by: Ran Shidlansik <ranshid@amazon.com>
ranshid added a commit that referenced this pull request Jan 29, 2026
This caused some unaligned load of server values on 32bit compilations.

Example:
https://github.com/valkey-io/valkey/actions/runs/21352969942/job/61491923381?pr=3111#step:6:3064

Some insights here:
#3111 (comment)

Signed-off-by: Ran Shidlansik <ranshid@amazon.com>
harrylin98 pushed a commit to harrylin98/valkey_forked that referenced this pull request Feb 19, 2026
This caused some unaligned load of server values on 32bit compilations.

Example:
https://github.com/valkey-io/valkey/actions/runs/21352969942/job/61491923381?pr=3111#step:6:3064

Some insights here:
valkey-io#3111 (comment)

Signed-off-by: Ran Shidlansik <ranshid@amazon.com>
hpatro pushed a commit to hpatro/valkey that referenced this pull request Mar 5, 2026
This caused some unaligned load of server values on 32bit compilations.

Example:
https://github.com/valkey-io/valkey/actions/runs/21352969942/job/61491923381?pr=3111#step:6:3064

Some insights here:
valkey-io#3111 (comment)

Signed-off-by: Ran Shidlansik <ranshid@amazon.com>
Signed-off-by: Harkrishn Patro <bunty.hari@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 9.0.2
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants