Skip to content

feat(storage): add sample for hierarchical namespace recursive delete - #16402

Open
nidhiii-27 wants to merge 6 commits into
mainfrom
feat/add-cpp-delete-folder-recursive
Open

nidhiii-27 wants to merge 6 commits into
mainfrom
feat/add-cpp-delete-folder-recursive

Conversation

@nidhiii-27

Copy link
Copy Markdown
Contributor

Draft PR for hierarchical namespace recursive delete folder sample.

[Generated-by: AI]

Adds a C++ code sample demonstrating hierarchical namespace recursive folder delete.

Fixes: b/530058946

[Generated-by: AI]
@product-auto-label product-auto-label Bot added api: storage Issues related to the Cloud Storage API. samples Issues that are directly related to samples. labels Aug 29, 2026
@nidhiii-27 nidhiii-27 added the storage-sample-architect Generated by storage-sample-architect skill label Aug 29, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

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 function DeleteFolderRecursive to demonstrate and test recursive folder deletion in the Storage Control service. It also updates the AutoRun integration test and the command-line entry points to include this new functionality. Feedback on the code suggests explicitly declaring the return type instead of using auto when calling DeleteFolderRecursive().get(), in accordance with the repository's style guide regarding obscured return types.

Comment thread google/cloud/storagecontrol/v2/samples/storage_control_folder_samples.cc Outdated
@codecov

codecov Bot commented Aug 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.31%. Comparing base (4955812) to head (4360492).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #16402      +/-   ##
==========================================
+ Coverage   92.29%   92.31%   +0.01%     
==========================================
  Files        2246     2246              
  Lines      213751   213751              
==========================================
+ Hits       197290   197327      +37     
+ Misses      16461    16424      -37     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread google/cloud/storagecontrol/v2/samples/storage_control_folder_samples.cc Outdated
@nidhiii-27
nidhiii-27 marked this pull request as ready for review September 7, 2026 17:00
@nidhiii-27
nidhiii-27 requested review from a team as code owners September 7, 2026 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API. samples Issues that are directly related to samples. storage-sample-architect Generated by storage-sample-architect skill

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants