Skip to content

Prototype BigQuery CDC -> Anomaly detection python flex template.#3479

Open
claudevdm wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
claudevdm:bqmonitor
Open

Prototype BigQuery CDC -> Anomaly detection python flex template.#3479
claudevdm wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
claudevdm:bqmonitor

Conversation

@claudevdm
Copy link
Contributor

No description provided.

@claudevdm claudevdm force-pushed the bqmonitor branch 2 times, most recently from 7229c10 to 393de61 Compare March 11, 2026 17:44
@gemini-code-assist
Copy link

Warning

Gemini is experiencing higher than usual traffic and was unable to create the summary. Please try again in a few hours by commenting /gemini summary.

@claudevdm
Copy link
Contributor Author

/gemini summary

@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 7.69231% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.22%. Comparing base (7264b3a) to head (d0786b4).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
...loud/teleport/plugin/maven/TemplatesStageMojo.java 0.00% 17 Missing and 1 partial ⚠️
...gle/cloud/teleport/plugin/DockerfileGenerator.java 25.00% 6 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##               main    #3479       +/-   ##
=============================================
+ Coverage     33.37%   52.22%   +18.85%     
- Complexity      481     6050     +5569     
=============================================
  Files           215     1040      +825     
  Lines         12816    62963    +50147     
  Branches       1249     6896     +5647     
=============================================
+ Hits           4277    32883    +28606     
- Misses         8203    27856    +19653     
- Partials        336     2224     +1888     
Components Coverage Δ
spanner-templates 72.23% <ø> (∅)
spanner-import-export 69.02% <ø> (∅)
spanner-live-forward-migration 80.49% <ø> (∅)
spanner-live-reverse-replication 77.88% <ø> (∅)
spanner-bulk-migration 89.18% <ø> (∅)
gcs-spanner-dv 85.34% <ø> (∅)
Files with missing lines Coverage Δ
...gle/cloud/teleport/plugin/DockerfileGenerator.java 85.59% <25.00%> (-4.41%) ⬇️
...loud/teleport/plugin/maven/TemplatesStageMojo.java 17.53% <0.00%> (-0.37%) ⬇️

... and 847 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gemini-code-assist
Copy link

Warning

Gemini is experiencing higher than usual traffic and was unable to create the summary. Please try again in a few hours by commenting /gemini summary.

@claudevdm
Copy link
Contributor Author

/gemini review

Copy link

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

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 BigQuery Anomaly Detection template, which is a significant and valuable addition. The changes are well-structured, covering both Java and Python components, and include comprehensive unit and integration tests. The safe_eval module is particularly well-designed for secure expression evaluation, and the updates to Dockerfile generation enhance the flexibility of Python template deployment. The new README provides excellent documentation for users. Overall, this is a high-quality contribution.

@claudevdm claudevdm changed the title initial Prototype BigQuery CDC -> Anomaly detection python flex template. Mar 11, 2026
@claudevdm claudevdm marked this pull request as ready for review March 12, 2026 17:27
@claudevdm
Copy link
Contributor Author

@shunping can you please take a look at the anomaly detection parts?

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant