Skip to content

Conversation

@alzimmermsft
Copy link
Member

@alzimmermsft alzimmermsft commented Jul 25, 2019

Contains new feature work done for Storage, the changes are the following:

  • File and Queue clients
  • SAS token generation for Blobs
  • Removed RawClients from Blobs
  • Merging of BlobClientBuilders
  • Added missing convenience layer methods for Blob Containers

All development was reviewed in PRs committed to the branch.

gapra-msft and others added 11 commits July 12, 2019 11:57
* SAS implementation

* Fixed some minor formatting issues

* Fixed checkstyle problems and test issue
Removes RawClients from Storage Blobs
…to ContainerClient (#4376)

* Removed raw clients

* Added deleteContainer to StorageClient

* Added getAppendBlob with snapshot to ContainerClient
* Initial check in for storage queue
* Finished the restructure, refactor builder. Added sleep in record feature. Linting
Merges AppendBlobClientBuilder, BlobClientBuilder, BlockBlobClientBuilder, and PageBlobClientBuilder into a single builder class BlobClientBuilder. Additionally, JavaDoc comments for the other builder classes, ContainerClientBuilder and StorageAccountClientBuilder, were cleaned up and the way the endpoint is handled in builders was changed.
@alzimmermsft alzimmermsft added Storage Storage Service (Queues, Blobs, Files) Client This issue points to a problem in the data-plane of the library. labels Jul 25, 2019
@alzimmermsft alzimmermsft self-assigned this Jul 25, 2019
@alzimmermsft alzimmermsft changed the title Merging in Feature Work for Storage Feature Work for Storage Jul 25, 2019
@alzimmermsft alzimmermsft merged commit 570632c into master Jul 25, 2019
@alzimmermsft alzimmermsft deleted the storage-post-review1-dev branch July 26, 2019 21:53
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 this pull request may close these issues.

5 participants