Skip to content

[DCP Ingestion] Remove the Source node existence check#594

Open
gmechali wants to merge 6 commits into
datacommonsorg:masterfrom
gmechali:source
Open

[DCP Ingestion] Remove the Source node existence check#594
gmechali wants to merge 6 commits into
datacommonsorg:masterfrom
gmechali:source

Conversation

@gmechali

Copy link
Copy Markdown
Contributor

No description provided.

@codacy-production

codacy-production Bot commented Jun 25, 2026

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 high · 1 medium

Alerts:
⚠ 2 issues (≤ 0 issues of at least minor severity)

Results:
2 new issues

Category Results
ErrorProne 1 high
Complexity 1 medium

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

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.

@gmechali gmechali requested a review from vish-cs June 25, 2026 21:49

@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 simplifies metadata validation by removing the requirement that a linked Source node must be explicitly defined in the database triples, now only verifying that each defined Provenance has a linked Source. The review feedback recommends sorting the defined provenances during iteration to ensure deterministic error messages and updating the class-level docstring of MetadataValidator to reflect these changes.

Comment thread simple/stats/validation.py Outdated
Comment thread simple/stats/validation.py
@gmechali gmechali enabled auto-merge (squash) June 26, 2026 14:02
@gmechali gmechali disabled auto-merge June 26, 2026 14:02
gmechali and others added 2 commits June 26, 2026 10:03
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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