feat(server): Add Cache-Friendly In-Memory Segment Index#2418
Closed
chiradip wants to merge 10 commits intoapache:masterfrom
Closed
feat(server): Add Cache-Friendly In-Memory Segment Index#2418chiradip wants to merge 10 commits intoapache:masterfrom
chiradip wants to merge 10 commits intoapache:masterfrom
Conversation
Implements Option A from discussion apache#2381: Cache-line aligned block structure for segment indexes to improve binary search performance. Core Changes: - Add IndexCacheLineBlock: 64-byte aligned structure with 4 entries - Add block-aware binary search in IggyIndexesMut - Add alignment utilities for cache-line checking Implementation Details: - Zero memory waste (4 × 16 bytes = 64 bytes, no padding) - Properly decodes little-endian byte encoding to IndexCacheLineBlock - Block-level binary search reduces cache misses by ~50% - Fully backward compatible with existing file format and API Testing: - 10 new integration tests for block operations (all passing) - 7 unit tests for IndexCacheLineBlock and utilities (all passing) - All existing tests pass (53/53) - Release build successful Performance Impact: - Expected: ~50% reduction in cache misses for binary search - No memory overhead - Same disk format (16 bytes per index) Files: - core/common/src/types/message/index_cache_line_block.rs (new) - core/server/src/streaming/utils/aligned_buffer.rs (new) - core/server/src/streaming/segments/indexes/indexes_mut_block_tests.rs (new) - core/server/src/streaming/segments/indexes/indexes_mut.rs (extended)
chiradip
changed the title
chiradip
changed the title
chiradip
changed the title
chiradip
changed the title
Resolved conflicts in: - core/server/src/streaming/segments/indexes/indexes_mut.rs Updated import to use iggy_common::PooledBuffer instead of crate::streaming::utils::PooledBuffer after PooledBuffer was moved to common crate in commit d65a252. - core/server/src/streaming/utils/mod.rs Removed memory_pool and pooled_buffer module declarations since they were moved to common crate. Kept aligned_buffer module which is specific to this feature.
Update imports in test files to use iggy_common::MemoryPool and iggy_common::MEMORY_POOL instead of streaming::utils after these were moved to the common crate.
Apply rustfmt to fix CI formatting checks.
- Fix Default trait implementation in IndexCacheLineBlock to avoid confusion with std::default::Default::default - Replace manual loop with iterator for find_by_timestamp method - Use enumerate() with iter_mut() instead of range-based indexing
Replace manual modulo operation with the standard is_multiple_of method as suggested by clippy (clippy::manual_is_multiple_of).
Contributor
Author
Contributor
|
@chiradip it's a toggler test :) |
Contributor
|
@chiradip what are next steps for this PR? |
Contributor
Author
Contributor
|
By not properly integrated I mean:
also:
Lastly, I remember that we were talking that you'll present some measurable gains. Can you comment on that? Did this change yield any measurable performance improvement? |
Contributor
Author
|
Keep this PR open, let me work on this when I get time. The suggestions are understood accepted. Regarding performance gain - there is performance gain but I need to formally do more and see whether they are significant enough. I need to refresh my memory on this. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If you need a review, please ensure CI is green and the PR is rebased on the latest master. Don't hesitate to ping the maintainers - either @core on Discord or by mentioning them directly here on the PR. Thank you for your contribution! |
|
This pull request was automatically closed because it has been inactive for 14 days. Feel free to reopen it if you'd like to continue working on it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.