Skip to content
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

CurveFS consider merging associated operations on dentry and parentInode #2548

Closed
SeanHai opened this issue Jun 21, 2023 · 2 comments
Closed

CurveFS consider merging associated operations on dentry and parentInode #2548

SeanHai opened this issue Jun 21, 2023 · 2 comments
Assignees
@201341
Labels
assigned enhancement improve feature repairing bug repairing

Comments

@SeanHai
Copy link
Contributor

SeanHai commented Jun 21, 2023

Is your feature request related to a problem? (你需要的功能是否与某个问题有关?)

The dentry and parenInode is on the same partition, The associated operations on dentry and parentInode maybe can merge. For example, when create dentry need update parentInode's time and other attrs. Now It is now implemented through multiple rpc.

Describe the solution you'd like (描述你期望的解决方法)

Describe alternatives you've considered (描述你想到的折衷方案)

Additional context/screenshots (更多上下文/截图)
@SeanHai SeanHai added the enhancement improve feature label Jun 21, 2023
@wuhongsong wuhongsong mentioned this issue Jul 3, 2023
Closed
@201341 201341 mentioned this issue Aug 7, 2023
Merged
2 tasks
@wuhongsong wuhongsong added the repairing bug repairing label Sep 8, 2023
@201341 201341 mentioned this issue Sep 12, 2023
Merged
2 tasks
@wuhongsong
Copy link
Contributor

work for it now

@caoxianfei1
Copy link
Contributor

Close it because the related pr #2744 has merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@201341 201341

Labels
assigned enhancement improve feature repairing bug repairing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@201341 @SeanHai @wuhongsong @caoxianfei1