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

GH-40025: [C++][FS][Azure] Implement Move() for flat namespace storage accounts #40203

Closed
wants to merge 4 commits into from

Conversation

felipecrv
Copy link
Contributor

@felipecrv felipecrv commented Feb 22, 2024

Rationale for this change

Move is currently only supported on HNS-enabled storage accounts.

What changes are included in this PR?

Implementation of all the checks + a TODO on the actual blob moves.

Are these changes tested?

  • By the same tests that cover the HNS storage account Move (pending removal of a conditional)

@felipecrv
Copy link
Contributor Author

@Tom-Newton Not fully implemented yet, but most of the annoying checks are out of the way.

@felipecrv
Copy link
Contributor Author

As explained in issue #40405 I decided to not continue the work on this operation and proposed a PR that returns a better error messages to users trying to use the unsupported Move (#40406). I will keep the branch available for anyone that wants to continue the work.

@felipecrv felipecrv closed this Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant