Skip to content

fs: fix compile error on latest RocksDB#300

Merged
yhr merged 1 commit into
westerndigitalcorporation:masterfrom
yhr:fssize_compile_fix
Feb 3, 2025
Merged

fs: fix compile error on latest RocksDB#300
yhr merged 1 commit into
westerndigitalcorporation:masterfrom
yhr:fssize_compile_fix

Conversation

@yhr

@yhr yhr commented Jan 21, 2025

Copy link
Copy Markdown
Collaborator

WritableFile::GetFileSize needs to be imlemented now, so hook it up to make it compilable on latest rocksdb.

WritableFile::GetFileSize needs to be imlemented now,
so hook it up to make it compilable on latest rocksdb.

Signed-off-by: Hans Holmberg <hans.holmberg@wdc.com>
@yhr yhr force-pushed the fssize_compile_fix branch from 3e44af2 to cfafae3 Compare January 21, 2025 14:01
@MaisenbacherD

Copy link
Copy Markdown
Contributor

Looks good to me :)
This standalone PR is better than burying the commit in PR #294 2230e12

@yhr

yhr commented Feb 3, 2025

Copy link
Copy Markdown
Collaborator Author

Looks like tests complete just fine but CI data collection is missbehaving, pulling this in

@yhr yhr merged commit 919c2eb into westerndigitalcorporation:master Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants