Skip to content

[DCP Ingestion] Ensure unique path for JSONLD output#512

Merged
gmechali merged 5 commits into
datacommonsorg:masterfrom
gmechali:uniquePath
May 21, 2026
Merged

[DCP Ingestion] Ensure unique path for JSONLD output#512
gmechali merged 5 commits into
datacommonsorg:masterfrom
gmechali:uniquePath

Conversation

@gmechali
Copy link
Copy Markdown
Contributor

This guarantees different ingestion runs will not impact each other

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 modifies simple/stats/runner.py to export JSON-LD files into unique subdirectories named after the import and a timestamp. Feedback suggests improving the robustness of the directory naming by using UTC timestamps with microsecond precision to prevent collisions and ensuring the import name defaults correctly if the configuration value is null.

Comment thread simple/stats/runner.py
Comment thread simple/stats/runner.py Outdated
@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented May 21, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 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 review from SandeepTuniki and vish-cs May 21, 2026 13:01
gmechali and others added 2 commits May 21, 2026 06:10
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@gmechali gmechali merged commit 58da713 into datacommonsorg:master May 21, 2026
6 checks passed
@gmechali gmechali deleted the uniquePath branch May 21, 2026 14:20
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