Skip to content

feat(bigquery): add dataplex_delete_data_profile_scan sample to delet… - #14716

Merged
iennae merged 1 commit into
mainfrom
feat/bq-dataplex-delete-data-profile-scan
Nov 3, 2026
Merged

iennae merged 1 commit into
mainfrom
feat/bq-dataplex-delete-data-profile-scan

Conversation

@iennae

@iennae iennae commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

…e a Dataplex data profile scan

Problem:
The Dataplex documentation (https://cloud.google.com/dataplex/docs/use-data-profiling#delete-scans) needs a standalone, tested Python code sample demonstrating how to delete a Dataplex data profile scan (region tag dataplex_delete_data_profile_scan).

Solution:

  • Add delete_data_profile_scan(project_id: str, location: str, data_scan_id: str) -> None in bigquery/data_profile_scan/dataplex_delete_data_profile_scan.py (Delete a Dataplex data profile scan resource using DataScanServiceClient.delete_data_scan and await completion.)
  • Add unit tests (bigquery/data_profile_scan/dataplex_delete_data_profile_scan_test.py) covering both the success path and API error handling (GoogleAPICallError)

Checklist

Testing

  • I have tested this change on a live environment and verified it works as intended.

Compliance & Style


Post-Approval Actions

  • Please merge this PR for me once it is approved

@iennae
iennae requested review from a team and chalmerlowe as code owners November 1, 2026 20:21
@snippet-bot

snippet-bot Bot commented Nov 1, 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

@product-auto-label product-auto-label Bot added api: bigquery Issues related to the BigQuery API. samples Issues that are directly related to samples. labels Nov 1, 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 introduces a new Python script to delete a Dataplex data profile scan resource, along with its corresponding unit tests and configuration files. The feedback suggests updating the noxfile_config.py file to ignore intermediate Python versions (3.11 and 3.12) and only test the oldest (3.10) and newest (3.13) supported releases, in accordance with repository guidelines.

@XrossFox XrossFox 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.

LGTM.

@iennae
iennae force-pushed the feat/bq-dataplex-delete-data-profile-scan branch 3 times, most recently from c2732d1 to 916d585 Compare November 2, 2026 23:33
…e a Dataplex data profile scan

Problem:
The Dataplex documentation (https://cloud.google.com/dataplex/docs/use-data-profiling#delete-scans)
needs a standalone, tested Python code sample demonstrating how to
delete a Dataplex data profile scan (region tag `dataplex_delete_data_profile_scan`).

Solution:
- Add `delete_data_profile_scan(project_id: str, location: str, data_scan_id: str) -> None` in `bigquery/data_profile_scan/dataplex_delete_data_profile_scan.py`
  (Delete a Dataplex data profile scan resource using DataScanServiceClient.delete_data_scan and await completion.)
- Add unit tests (`bigquery/data_profile_scan/dataplex_delete_data_profile_scan_test.py`) covering both the success
  path and API error handling (`GoogleAPICallError`)
@iennae
iennae force-pushed the feat/bq-dataplex-delete-data-profile-scan branch from 916d585 to ce061f6 Compare November 3, 2026 02:44
@iennae
iennae merged commit 7f869c7 into main Nov 3, 2026
14 of 22 checks passed
@iennae
iennae deleted the feat/bq-dataplex-delete-data-profile-scan branch November 3, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the BigQuery API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants