Skip to content

[Storage] STG105 Modify delete container documentation - #50853

Merged
browndav-msft merged 1 commit into
Azure:feature/storage/stg105basefrom
ibrandes:stg105/deleteContainerDocs
Oct 26, 2026
Merged

browndav-msft merged 1 commit into
Azure:feature/storage/stg105basefrom
ibrandes:stg105/deleteContainerDocs

Conversation

@ibrandes

Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions Bot added the Storage Storage Service (Queues, Blobs, Files) label Oct 18, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
34 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@ibrandes
Isabelle (ibrandes) changed the base branch from main to feature/storage/stg105base October 18, 2026 18:07
@ibrandes
Isabelle (ibrandes) requested a balanced review from Copilot October 18, 2026 18:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Service-owner confirmation is needed for the documented 30-second access window and subsequent 404 guarantee.

No open issues 🎉

What changed in this PR

Updates Azure Blob Storage and Data Lake deletion documentation without changing runtime behavior.

Changes:

  • Describes delayed name reuse and temporary access after deletion.
  • Applies consistent JavaDoc across synchronous and asynchronous clients.
  • Records the documentation updates in both package changelogs.
File Description
sdk/​storage/​azure-storage-file-datalake/​src/​main/​java/​com/​azure/​storage/​file/​datalake/​DataLakeServiceClient.java Documents synchronous file-system deletion behavior.
sdk/​storage/​azure-storage-file-datalake/​src/​main/​java/​com/​azure/​storage/​file/​datalake/​DataLakeServiceAsyncClient.java Documents asynchronous file-system deletion behavior.
sdk/​storage/​azure-storage-file-datalake/​src/​main/​java/​com/​azure/​storage/​file/​datalake/​DataLakeFileSystemClient.java Updates synchronous delete overloads.
sdk/​storage/​azure-storage-file-datalake/​src/​main/​java/​com/​azure/​storage/​file/​datalake/​DataLakeFileSystemAsyncClient.java Updates asynchronous delete overloads.
sdk/​storage/​azure-storage-file-datalake/​CHANGELOG.md Records Data Lake documentation changes.
sdk/​storage/​azure-storage-blob/​src/​main/​java/​com/​azure/​storage/​blob/​BlobServiceClient.java Documents synchronous container deletion behavior.
sdk/​storage/​azure-storage-blob/​src/​main/​java/​com/​azure/​storage/​blob/​BlobServiceAsyncClient.java Documents asynchronous container deletion behavior.
sdk/​storage/​azure-storage-blob/​src/​main/​java/​com/​azure/​storage/​blob/​BlobContainerClient.java Updates synchronous delete overloads.
sdk/​storage/​azure-storage-blob/​src/​main/​java/​com/​azure/​storage/​blob/​BlobContainerAsyncClient.java Updates asynchronous delete overloads.
sdk/​storage/​azure-storage-blob/​CHANGELOG.md Records Blob Storage documentation changes.

Review effort: Balanced

@browndav-msft browndav-msft left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Everything here looks good.

I don't think it is necessary, but we are stating a service side contract that a same-name recreation before 30 seconds is a 409 ContainerBeingDeleted and an immediate child read is a 404 ContainerNotFound, which are both correct. And then a 201 Created

Since we are making this claim for the SDK, I think it might help to have a test for this to make sure that what we are claiming for the SDK is true, although it is a service level agreement.

But not necessary.

@browndav-msft
browndav-msft merged commit 1bddf63 into Azure:feature/storage/stg105base Oct 26, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants