Skip to content

only send header once per peer per group and fix tests for a green ra…#16613

Open
aarshkshah1992 wants to merge 1 commit intofeat/cleanup-logs-dont-construct-partial-columns-if-disabledfrom
feat/only-send-header-once-per-peer-per-group
Open

only send header once per peer per group and fix tests for a green ra…#16613
aarshkshah1992 wants to merge 1 commit intofeat/cleanup-logs-dont-construct-partial-columns-if-disabledfrom
feat/only-send-header-once-per-peer-per-group

Conversation

@aarshkshah1992
Copy link
Copy Markdown
Contributor

@aarshkshah1992 aarshkshah1992 commented Mar 31, 2026 •
edited
Loading

Closes #16580.

What type of PR is this?
Optimisation

What does this PR do? Why is it needed?

  • This PR ensures we only push the header once per peer per group (block root) across all columns.
  • It also ensures that all the partial data column tests are green when race detector is enabled.

Acknowledgements

  • I have read CONTRIBUTING.md.
  • I have included a uniquely named changelog fragment file.
  • I have added a description with sufficient context for reviewers to understand this PR.
  • I have tested that my changes work as expected and I added a testing plan to the PR description (if applicable).

@aarshkshah1992 aarshkshah1992 requested a review from kasey March 31, 2026 14:34
@aarshkshah1992 aarshkshah1992 self-assigned this Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kasey kasey Awaiting requested review from kasey

Assignees

@aarshkshah1992 aarshkshah1992

Labels

partial-cells-getBlobs-v3

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@aarshkshah1992