-
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
Load content hash as the etag of the object when the UFS is S3 #18440
Load content hash as the etag of the object when the UFS is S3 #18440
Conversation
Automated checks report:
Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks. |
Automated checks report:
All checks passed! |
alluxio-bot, merge this please |
alluxio-bot, cherry-pick this to branch-2.10 please |
Auto cherry-pick unsuccessful Failed to setup local git for auto cherry-pick 91e045b cannot be auto cherry-picked to branch branch-2.10. Please cherry-pick this manually. |
Recode the content hash into XAttr as the ETag when the UFS is S3. When the file is newly created or loaded from UFS, Alluxio will record the ContentHash into metastore so the user can see the ETag attribute. When the UFS is Object storage, The user can see the ETag of the file persisted in UFS by this way. No pr-link: Alluxio#18440 change-id: cid-1204732240a5e73959b917eb6d9f0c97e05820dc
Cherry-pick of existing commit. orig-pr: Alluxio#18440 orig-commit: Alluxio/alluxio@91e045b orig-commit-author: yuyang wang <39869597+Jackson-Wang-7@users.noreply.github.com> pr-link: Alluxio#18446 change-id: cid-1204732240a5e73959b917eb6d9f0c97e05820dc
What changes are proposed in this pull request?
Recode the content hash into XAttr as the ETag when the UFS is S3. When the file is newly created or loaded from UFS, Alluxio will record the ContentHash into metastore so the user can see the ETag attribute.
Why are the changes needed?
When the UFS is Object storage, The user can see the ETag of the file persisted in UFS by this way.
Does this PR introduce any user facing changes?
No