Skip to content

feat(storage/control): add delete folder recursive sample - #14397

Open
nidhiii-27 wants to merge 2 commits into
mainfrom
feat/storage-delete-folder-recursive-sample
Open

nidhiii-27 wants to merge 2 commits into
mainfrom
feat/storage-delete-folder-recursive-sample

Conversation

@nidhiii-27

Copy link
Copy Markdown
Contributor

Re-applies the changes from #13642 (which was reverted in #14391 due to #14387) and includes librarian generation updates so that the librarian generate diff check succeeds.

[Generated-by: AI]

Re-applies the changes from #13642 (which was reverted in #14391 due to #14387) and includes librarian generation updates so that the librarian generate diff check succeeds.

[Generated-by: AI]
@nidhiii-27 nidhiii-27 added ai-generated storage-sample-architect Generated by storage-sample-architect skill labels Sep 16, 2026

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request adds a new sample and test for recursively deleting folders using the Storage Control API. The review feedback recommends removing the 30-second timeout on the long-running recursive delete operation to avoid premature timeout exceptions, which also allows simplifying the method signatures by removing the unused TimeoutException.

@nidhiii-27
nidhiii-27 marked this pull request as ready for review September 16, 2026 06:55
@nidhiii-27
nidhiii-27 requested review from a team as code owners September 16, 2026 06:55
@snippet-bot

snippet-bot Bot commented Sep 16, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 1 region tag.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

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

Labels

ai-generated storage-sample-architect Generated by storage-sample-architect skill

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant