alluxio icon indicating copy to clipboard operation
alluxio copied to clipboard

Fix fuse metadata cache outdated issue after chmod/chown

Open secfree opened this issue 1 year ago • 3 comments

What changes are proposed in this pull request?

Fix fuse metadata cache outdated issue after chmod/chown

Why are the changes needed?

Fix #18523

Does this PR introduce any user facing changes?

No

secfree avatar Feb 20 '24 08:02 secfree

I think there's a situation here that needs consideration: if the directory is large, caching it in the metadata cache would take a long time. Is it reasonable to completely invalidate the cache of the parent directory when modifications occur in the child nodes, or should we simply replace the corresponding child nodes in the list cache?

humengyu2012 avatar Feb 28 '24 05:02 humengyu2012

@jiacheliu3 can you take a look?

humengyu2012 avatar Feb 28 '24 05:02 humengyu2012

I think there's a situation here that needs consideration: if the directory is large, caching it in the metadata cache would take a long time. Is it reasonable to completely invalidate the cache of the parent directory when modifications occur in the child nodes, or should we simply replace the corresponding child nodes in the list cache?

@humengyu2012 thanks for your reply. Currently MetadataCachingFileSystem only syncs metadata from the cluster but not replace existing ones. So I just keep setAcl/setAttribute consistent with createFile/createDirectory/delete/rename here.

secfree avatar Mar 01 '24 09:03 secfree