Skip to content

This change added the last_update_timestamp to the integration test to reflect the schema change that includes the new field#514

Merged
shixiao-coder merged 2 commits into
datacommonsorg:masterfrom
shixiao-coder:fix-integration-test-failure-for-missing-timestamp
May 27, 2026
Merged

This change added the last_update_timestamp to the integration test to reflect the schema change that includes the new field#514
shixiao-coder merged 2 commits into
datacommonsorg:masterfrom
shixiao-coder:fix-integration-test-failure-for-missing-timestamp

Conversation

@shixiao-coder
Copy link
Copy Markdown
Contributor

Integration test failed in https://pantheon.corp.google.com/cloud-build/builds;region=global/4ff57037-7aae-4166-ac2d-914809747f75;step=1?e=13803378&mods=-monitoring_api_staging&project=datcom-ci, it dues to the test schema.sql not updated for the integration test.

This PR updated this test schema.sql

Copy link
Copy Markdown
Contributor

@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 adds a last_update_timestamp column to the Node table in the Spanner test schema. The reviewer noted that this column is missing from the main production schema file, which could lead to schema drift, and requested clarification on how schema changes are applied to existing databases.

Comment thread pipeline/ingestion/src/test/resources/spanner_schema.sql
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@shixiao-coder shixiao-coder requested review from gmechali and vish-cs May 27, 2026 18:50
Copy link
Copy Markdown
Contributor

@gmechali gmechali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks Xiao! it would defintely be very nice to get those merged.

@shixiao-coder shixiao-coder merged commit 89d000b into datacommonsorg:master May 27, 2026
10 checks passed
@shixiao-coder shixiao-coder deleted the fix-integration-test-failure-for-missing-timestamp branch May 27, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants