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

Copy local files should support hdfs semantics #18470

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lubo212
Copy link

@lubo212 lubo212 commented Dec 18, 2023

What changes are proposed in this pull request?

copy local files should support hdfs semantics

Why are the changes needed?

If there is such a local path that needs to be uploaded:
image

I think users want to see something like this after executing the cp command:
image

If I upload an empty directory, I want it to be allowed:
image

@alluxio-bot
Copy link
Contributor

Automated checks report:

  • PR title follows the conventions: FAIL
    • The title of the PR does not pass all the checks. Please fix the following issues:
      • First word must be capitalized
  • Commits associated with Github account: FAIL
    • It looks like your commits can't be linked to a valid Github account.
      Your commits are made with the email bollu@tencent.com, which does not allow your contribution to be tracked by Github.
      See this link for possible reasons this might be happening.
      To change the author email address that your most recent commit was made under, you can run:
      git -c user.name="Name" -c user.email="Email" commit --amend --reset-author
      See this answer for more details about how to change commit email addresses.
      Once the author email address has been updated, update the pull request by running:
      git push --force https://github.com/lubo212/alluxio.git localCopySupportHdfs

Some checks failed. Please fix the reported issues and reply 'alluxio-bot, check this please' to re-run checks.

@lubo212 lubo212 changed the title copy local files should support hdfs semantics Copy local files should support hdfs semantics Dec 18, 2023
@alluxio-bot
Copy link
Contributor

Automated checks report:

  • PR title follows the conventions: PASS
  • Commits associated with Github account: PASS

All checks passed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants