Skip to content

Class MemFS inherits FilesystemBase. #5573

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.

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

Merged
merged 6 commits into from
Jul 9, 2025
Merged

Conversation

bekadavis9
Copy link
Contributor

Class MemFS inherits base class FilesystemBase.

This is the first PR in the VFS refactor work. The primary goal is for MemFS to inherit the base class, FilesystemBase. Some additional changes were made to the base class and to VFS to ease this transition for all other filesystems.


TYPE: IMPROVEMENT
DESC: Class MemFS inherits base class FilesystemBase.


Resolves CORE-289

@bekadavis9 bekadavis9 requested a review from teo-tsirpanis July 7, 2025 16:23
@bekadavis9 bekadavis9 force-pushed the rd/vfs_refactor-memfs branch from 79da456 to 1e296c7 Compare July 7, 2025 17:38
@bekadavis9 bekadavis9 requested a review from teo-tsirpanis July 8, 2025 17:27
@bekadavis9 bekadavis9 force-pushed the rd/vfs_refactor-memfs branch from c0117f0 to 67d9a08 Compare July 8, 2025 17:54
Copy link
Member

@teo-tsirpanis teo-tsirpanis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

*/
void touch_internal(const std::string& path, FSNode** node = nullptr) const;
FSNode* touch_internal(const std::string& path) const;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: these functions already track the returned nodes in a tdb_unique_ptr, so returning a pointer is OK.

@bekadavis9 bekadavis9 force-pushed the rd/vfs_refactor-memfs branch from 67d9a08 to 9294d62 Compare July 8, 2025 19:30
@bekadavis9 bekadavis9 force-pushed the rd/vfs_refactor-memfs branch from 9294d62 to 4da7966 Compare July 8, 2025 21:14
@bekadavis9 bekadavis9 merged commit c2b5e90 into main Jul 9, 2025
56 checks passed
@bekadavis9 bekadavis9 deleted the rd/vfs_refactor-memfs branch July 9, 2025 01:40
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