Skip to content

feat[backend](ingestion_stats): fixed default time window for ingesti… - #2407

Merged
AlexSanchez-bit merged 1 commit into
release/v12.0.0from
backlog/v12_ingestion_stats
Jul 29, 2026
Merged

feat[backend](ingestion_stats): fixed default time window for ingesti…#2407
AlexSanchez-bit merged 1 commit into
release/v12.0.0from
backlog/v12_ingestion_stats

Conversation

@AlexSanchez-bit

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown

🛑 AI review — High/critical findings

One or more high/critical issues were found. Please review and fix before merging if they're real.

architecture (gemini-3-flash-lite) — clean

Summary: Refactored defaultStatsWindow constant from time.Duration to string within eventprocessing usecase.

No findings.

🛑 bugs (gemini-3-flash-lite) — high/critical — please review

Summary: Changed defaultStatsWindow constant from time.Duration to string, causing type mismatch in resolveWindow.

  • high backend/modules/eventprocessing/usecase/ingestion_stats.go:16 — defaultStatsWindow was changed from a time.Duration to a string ("24h"). If it is used elsewhere in the codebase as a time.Duration, this will cause compilation errors or type mismatches. Ensure all usages of defaultStatsWindow are updated or revert the type change.

security (gemini-3-flash-lite) — clean

Summary: Routine refactoring of event processing ingestion stats window constant from time.Duration to string.

No findings.

🟢 go-deps — up to date

No pending Go dependency updates.

@AlexSanchez-bit

Copy link
Copy Markdown
Contributor Author

that constant is only used there

@AlexSanchez-bit
AlexSanchez-bit merged commit 193c86a into release/v12.0.0 Jul 29, 2026
1 check passed
@AlexSanchez-bit
AlexSanchez-bit deleted the backlog/v12_ingestion_stats branch July 29, 2026 17:34
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.

1 participant