Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adjust logging level for JDBC resource leak warnings #45583

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

gastaldi
Copy link
Contributor

```
- Adjust JDBC resource leak logging to use debug level instead of warnings for better log granularity.
- Enhance developer experience by reducing noise in application logs.
```
@gastaldi
Copy link
Contributor Author

/cc @mabartos

Copy link
Member

@yrodiere yrodiere left a comment

Choose a reason for hiding this comment

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

Thanks!

@gastaldi gastaldi added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jan 14, 2025
Copy link
Contributor

@mabartos mabartos left a comment

Choose a reason for hiding this comment

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

@gastaldi Great, thanks!

Copy link

quarkus-bot bot commented Jan 14, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 81a7c4b.

Failing Jobs

Status Name Step Failures Logs Raw logs Build scan
✔️ JVM Tests - JDK 17 Logs Raw logs 🔍
✔️ JVM Tests - JDK 21 Logs Raw logs 🔍
JVM Tests - JDK 17 Windows Clean Gradle temp directory ⚠️ Check → Logs Raw logs 🔍

You can consult the Develocity build scans.

@gastaldi gastaldi merged commit bbf1153 into quarkusio:main Jan 14, 2025
42 of 43 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.18 - main milestone Jan 14, 2025
@quarkus-bot quarkus-bot bot added kind/enhancement New feature or request and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels Jan 14, 2025
@gastaldi gastaldi deleted the agroal_logging branch January 14, 2025 18:54
@gsmet gsmet modified the milestones: 3.18 - main, 3.17.7 Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Demote Agroal's "JDBC resources leaked" warnings to DEBUG
4 participants