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.

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

Return relative path in listStatus results #18014

Merged
merged 6 commits into from
Aug 22, 2023

Conversation

dbw9580
Copy link
Contributor

@dbw9580 dbw9580 commented Aug 17, 2023

What changes are proposed in this pull request?

Return relative path in listStatus results.

Why are the changes needed?

After #17839 the worker only receives and sends absolute UFS paths. This change convert them back to Alluxio relative path according to client's perspective about the UFS.

Does this PR introduce any user facing changes?

No.

Copy link
Contributor

@huanghua78 huanghua78 left a comment

Choose a reason for hiding this comment

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

Looks good.

@dbw9580 dbw9580 added the type-feature This issue is a feature request label Aug 22, 2023
@dbw9580
Copy link
Contributor Author

dbw9580 commented Aug 22, 2023

alluxio-bot, merge this please

@alluxio-bot alluxio-bot merged commit 25a2c4a into Alluxio:main Aug 22, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature This issue is a feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants