Skip to content

Backport release/v6.4: Disable zero copy for SC#3108

Merged
masih merged 1 commit intorelease/v6.4from
backport-3107-to-release/v6.4
Mar 23, 2026
Merged

Backport release/v6.4: Disable zero copy for SC#3108
masih merged 1 commit intorelease/v6.4from
backport-3107-to-release/v6.4

Conversation

@seidroid
Copy link
Copy Markdown

@seidroid seidroid bot commented Mar 21, 2026

Backport of #3107 to release/v6.4.

## Describe your changes and provide context
This PR completely disable zero copy for SC to avoid segfault exception.
It also reverts #3098

## Testing performed to validate your change

---------

Co-authored-by: Masih H. Derkani <m@derkani.org>
(cherry picked from commit b1cb904)
@seidroid seidroid bot added the backport label Mar 21, 2026
@seidroid seidroid bot added the backport label Mar 21, 2026
@seidroid seidroid bot mentioned this pull request Mar 21, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 21, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMar 23, 2026, 12:30 PM

@github-actions
Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMar 21, 2026, 8:49 PM

@github-actions
Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMar 21, 2026, 8:50 PM

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.39%. Comparing base (0318a62) to head (1f5d94e).
⚠️ Report is 1 commits behind head on release/v6.4.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           release/v6.4    #3108      +/-   ##
================================================
- Coverage         58.39%   58.39%   -0.01%     
================================================
  Files              2088     2088              
  Lines            172135   172130       -5     
================================================
- Hits             100525   100520       -5     
  Misses            62647    62647              
  Partials           8963     8963              
Flag Coverage Δ
sei-chain-pr 68.03% <100.00%> (?)
sei-db 70.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
sei-cosmos/storev2/rootmulti/store.go 46.49% <ø> (-0.48%) ⬇️
sei-db/state_db/sc/memiavl/store.go 90.12% <100.00%> (ø)
🚀 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.

@masih masih enabled auto-merge (squash) March 21, 2026 20:57
@masih masih requested review from philipsu522 and sei-will March 23, 2026 12:30
@masih masih merged commit 8adc72e into release/v6.4 Mar 23, 2026
42 of 45 checks passed
@masih masih deleted the backport-3107-to-release/v6.4 branch March 23, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants