-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Wait for file completed in fuse getAttr (#8192) #9496
Conversation
Merged build finished. Test FAILed. |
Test FAILed. |
jenkins, test this please |
Merged build finished. Test FAILed. |
Test FAILed. |
|
|
||
URIStatus status = mFileSystem.getStatus(turi); | ||
if (!status.isCompleted()) { | ||
if (!waitForFileCompleted(turi)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we consider the mOpenFiles?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean to check if it is in mOpenFiles
? The file could be closed during the wait period.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
alluxio-bot, merge this please |
cherry-pick of #8192
cherry-pick commits are:
change fuse get attr : fb36eea63268
change the error message : 9c78eaa12b36
update status after file completed : 70eb188fa485
fix the getAttr error : c270a101fad6