-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[TieredStorage] Define OwnerOffset as u32 (#34105)
#### Problem The current OwnerOffset is defined as usize, which actual size could be u32 or u64. On the other hand, the OwnerOffset is used to access the ith owner in one TieredStorageFile. As a result, u32 is more than enough to represent the number of owners inside one TieredStorageFile while u16 might not be always enough. #### Summary of Changes This PR defines OwnerOffset as u32. #### Test Plan Existing test cases.
- Loading branch information
1 parent
3081b43
commit f598870
Showing
2 changed files
with
10 additions
and
5 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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