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

Add "lakectl fs upload --recursive" flag #979

Merged
merged 2 commits into from Dec 2, 2020

Conversation

arielshaqed
Copy link
Contributor

Behaves a lot like s3 cp --recursive: uploads files using their path relative from the local directory.

Example

Suppose you have these files:

/tmp/a/foo
/tmp/a/b/bar
/tmp/a/b/c/d/e/xyzzy

Then lakectl upload --recursive --source /tmp/a/ lakefs://repo@master/top/ will create these objects:

lakefs://repo@master/top/foo
lakefs://repo@master/top/b/bar
lakefs://repo@master/top/b/c/d/e/xyzzy

@arielshaqed
Copy link
Contributor Author

Alternative solution for https://lakefs.slack.com/archives/C016726JLJW/p1606735311131500.

@arielshaqed
Copy link
Contributor Author

Alternative to #977, probably don't want both...

Copy link
Collaborator

@ozkatz ozkatz left a comment

Choose a reason for hiding this comment

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

LGTM!

@arielshaqed
Copy link
Contributor Author

Thanks! Pulling this one, throwing #977 away.

@arielshaqed arielshaqed merged commit 3aa396f into master Dec 2, 2020
@arielshaqed arielshaqed deleted the feature/upload-recursive branch December 2, 2020 16:04
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