Skip to content

Reduce noisy WARN logs to DEBUG level#1269

Open
milanmajchrak wants to merge 1 commit intodtq-devfrom
propagate/reduce-warn-logs-dtq-dev
Open

Reduce noisy WARN logs to DEBUG level#1269
milanmajchrak wants to merge 1 commit intodtq-devfrom
propagate/reduce-warn-logs-dtq-dev

Conversation

@milanmajchrak
Copy link
Collaborator

Cherry-pick of PR #1263 into dtq-dev.

Problem description

Changed two frequently occurring WARN log messages to DEBUG level:

  • Context.java: 'Initializing a context while an active transaction exists'
  • ClarinItemServiceImpl.java: 'Cannot update item dates metadata because the approximate date is empty'

Original PR: #1263

Changed two frequently occurring WARN log messages to DEBUG level:
- Context.java: 'Initializing a context while an active transaction exists'
- ClarinItemServiceImpl.java: 'Cannot update item dates metadata because the approximate date is empty'
Copilot AI review requested due to automatic review settings March 13, 2026 11:45
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces log noise in the DSpace backend by downgrading two high-frequency log events from WARN to DEBUG, keeping production logs cleaner while still preserving diagnostic detail when DEBUG is enabled.

Changes:

  • Lowered Context initialization log about an active transaction from WARN to DEBUG.
  • Lowered ClarinItemServiceImpl log about missing approximate date metadata from WARN to DEBUG.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
dspace-api/src/main/java/org/dspace/core/Context.java Downgrades a frequent “active transaction exists” initialization log to DEBUG.
dspace-api/src/main/java/org/dspace/content/clarin/ClarinItemServiceImpl.java Downgrades a frequent “approximate date is empty” log to DEBUG.

You can also share your feedback on Copilot code review. Take the survey.

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