Skip to content

Speed up RDMA #4921

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
drbasic merged 5 commits into from
Jan 7, 2026
Merged

Speed up RDMA #4921

drbasic merged 5 commits into from
Jan 7, 2026

Conversation

@drbasic
Copy link
Collaborator

@drbasic drbasic commented Dec 30, 2025 •
edited
Loading

  1. Rename sentRequestId to clientRequestId
  2. Generate 16-bit reqId with sequential number generator
  3. History of completed, timed out, cancelled requests removed
  4. Speed up PopTimedOutRequests()
image

@drbasic drbasic added large-tests Launch large tests for PR blockstore Add this label to run only cloud/blockstore build and tests on PR labels Dec 30, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 1c51034.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
6096 6095 0 0 0 1 0

@drbasic drbasic marked this pull request as ready for review December 30, 2025 07:48
@drbasic drbasic added asan Launch builds with address sanitizer along with regular build tsan Launch builds with thread sanitizer along with regular build labels Dec 30, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-release-asan: all tests PASSED for commit 1702fb6.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
5446 5446 0 0 0 0 0

@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-release-tsan: some tests FAILED for commit 1702fb6.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
5413 5408 0 1 0 4 0

@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 1702fb6.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
6096 6095 0 0 0 1 0

tpashkin
tpashkin reviewed Dec 31, 2025
View reviewed changes
@drbasic drbasic merged commit 4876916 into main Jan 7, 2026
48 of 53 checks passed
@drbasic drbasic deleted the users/drbasic/speedup-rdma branch January 7, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tpashkin tpashkin tpashkin approved these changes

@komarevtsev-d komarevtsev-d komarevtsev-d approved these changes

Assignees

No one assigned

Labels

asan Launch builds with address sanitizer along with regular build blockstore Add this label to run only cloud/blockstore build and tests on PR large-tests Launch large tests for PR tsan Launch builds with thread sanitizer along with regular build

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@drbasic @tpashkin @komarevtsev-d