Skip to content

chore(video): remove discontinued AutoML samples - #10380

Draft
XrossFox wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
XrossFox:remove-video-autmoml-samples
Draft

XrossFox wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
XrossFox:remove-video-autmoml-samples

Conversation

@XrossFox

@XrossFox XrossFox commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Description

Remove discontinued AutoML samples

Remaining sample tests are disabled and will be reworked in b/584402119.

Fixes b/584400026

Checklist

Testing

  • I have tested this change on a live environment and verified it works as intended.
  • Tests pass: mvn clean verify required
  • Lint passes: mvn -P lint checkstyle:check required
  • Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only

Compliance & Style

  • I have followed Sample Format Guide
  • pom.xml parent set to latest shared-configuration
  • Appropriate changes to README are included in PR
  • This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
  • This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample

Post-Approval Actions

  • Please merge this PR for me once it is approved

@XrossFox
XrossFox requested review from a team and yoshi-approver as code owners November 10, 2026 23:41
@snippet-bot

snippet-bot Bot commented Nov 10, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to delete 3 region tags.

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 samples Issues that are directly related to samples. api: videointelligence Issues related to the Video Intelligence API API. labels Nov 10, 2026
@XrossFox
XrossFox marked this pull request as draft November 10, 2026 23:41

@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 deletes three beta video streaming AutoML sample files: StreamingAutoMlActionRecognition.java, StreamingAutoMlClassification.java, and StreamingAutoMlObjectTracking.java. The reviewer pointed out that the corresponding integration test files (StreamingAutoMlActionRecognitionIT.java, StreamingAutoMlClassificationIT.java, and StreamingAutoMlObjectTrackingIT.java) must also be deleted to prevent compilation errors and build failures.

Remaining sample tests are disabled and will be reworked in b/584402119.
@XrossFox
XrossFox force-pushed the remove-video-autmoml-samples branch from 5569f44 to c4a4276 Compare November 12, 2026 20:07
@XrossFox XrossFox changed the title WIP delete video automl samples chore(video): remove discontinued AutoML samples Nov 12, 2026

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

I found two files that are still in use; please undo the changes so we don't have broken links.

@XrossFox

Copy link
Copy Markdown
Contributor Author

I found two files that are still in use; please undo the changes so we don't have broken links.

There is a change pending review already to remove these docs.

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

Labels

api: videointelligence Issues related to the Video Intelligence API API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants