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

Implement a copy function for collections #496

Open
hechth opened this issue Nov 30, 2023 · 8 comments
Open

Implement a copy function for collections #496

hechth opened this issue Nov 30, 2023 · 8 comments

Comments

@hechth
Copy link

hechth commented Nov 30, 2023

It would be cool if collections could be copied the same way as it is currently possible for data objects using a single function call to the respective manager.

@trel
Copy link
Member

trel commented Nov 30, 2023

recursive calls are currently client-driven...

implementing server-side recursion might depend on something like... irods/irods#7213

@trel
Copy link
Member

trel commented Nov 30, 2023

icp handles this via cpUtil.cpp in irods/irods, which is part of irods-dev(el), which is available to the C++ client iCommands.

Currently, other languages have to handle this logic themselves... which is... not what we want for the long term. Hence a server-side solution would be nice/good.

@alanking
Copy link
Contributor

Whether or not it was ever meant to be used, we have had this thing sitting around for quite a long time... https://github.com/irods/irods/blob/aff93a3fbe417c8ec2e9d834b5eb2d6b2ae31083/plugins/experimental/src/copy.cpp#L40

@trel
Copy link
Member

trel commented Nov 30, 2023

but... is that available for use by a call from Python?

@alanking
Copy link
Contributor

It's an API plugin, so... yes?

@trel
Copy link
Member

trel commented Nov 30, 2023

Well, right, that's so smart.

@alanking
Copy link
Contributor

Hmm... an "experimental" plugin. Maybe not.

@d-w-moore
Copy link
Collaborator

d-w-moore commented Dec 1, 2023

Easily done in pure python too. Even easier if we use pyfilesystem2's fs.walk()

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

No branches or pull requests

4 participants