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

[Storage] [DataMovement] Refactor Try-style methods #46644

Closed
nickliu-msft opened this issue Oct 15, 2024 · 0 comments · Fixed by #46645
Closed

[Storage] [DataMovement] Refactor Try-style methods #46644

nickliu-msft opened this issue Oct 15, 2024 · 0 comments · Fixed by #46645
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files)

Comments

@nickliu-msft
Copy link
Member

nickliu-msft commented Oct 15, 2024

"We have a lot of TryXyz style methods where we just never actually check the resulting bool. That is super confusing for maintenance and could be hiding things. We need to go through those and determining which ones don't need to be try patterns vs which ones we need to react to failure will be very helpful"

@nickliu-msft nickliu-msft self-assigned this Oct 15, 2024
@jsquire jsquire added Storage Storage Service (Queues, Blobs, Files) Client This issue points to a problem in the data-plane of the library. labels Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants