Skip to content

Enable JUL logging for fat jar and update logging documentation#1168

Merged
jayantsing-db merged 5 commits intodatabricks:mainfrom
jayantsing-db:jayantsing-db/oss-jdbc-logging
Mar 2, 2026
Merged

Enable JUL logging for fat jar and update logging documentation#1168
jayantsing-db merged 5 commits intodatabricks:mainfrom
jayantsing-db:jayantsing-db/oss-jdbc-logging

Conversation

@jayantsing-db
Copy link
Copy Markdown
Collaborator

@jayantsing-db jayantsing-db commented Jan 5, 2026

Description

Replace Log4j2 dependencies with the SLF4J-to-JUL bridge (slf4j-jdk14) so
that the fat jar can emit SDK and Apache HTTP client logs via Java Util
Logging (JUL) without requiring users to supply additional logging
libraries.

  • Simplify JulLogger implementation to delegate directly to java.util.logging
  • Remove log4j-core, log4j-api, and log4j-slf4j2-impl test dependencies
  • Remove Log4j2 shade plugin transformers from pom.xml
  • Restructure docs/LOGGING.md with a fat jar vs thin jar quick-reference
    table, JUL configuration examples, and SLF4J/Log4j2/Logback integration
    guides
  • Simplify logging section in README.md to point to detailed docs

Testing

  • Unit tests
  • E2e tests

Additional Notes to the Reviewer

@jayantsing-db jayantsing-db changed the title Update Logging documentation Update JUL Logging and documentation Jan 7, 2026
@jayantsing-db
Copy link
Copy Markdown
Collaborator Author

Sample JUL logs for a query execution using the fat jar with the fix in this PR that includes SDK, apache http client logs: https://temp.corp.databricks.com/?4b6ff9f7fff43a0d#JAu6AHDbAPyzwmUHX3StVfRhmQDbtxhEKzijCLTLKzH4

@github-actions
Copy link
Copy Markdown

This PR has been marked as Stale because it has been open for 30 days with no activity. If you would like the PR to remain open, please remove the stale label or comment on the PR.

@jayantsing-db jayantsing-db changed the title Update JUL Logging and documentation Enable JUL logging for fat jar and update logging documentation Mar 2, 2026
@jayantsing-db jayantsing-db enabled auto-merge (squash) March 2, 2026 07:15
@jayantsing-db jayantsing-db merged commit 00ceea5 into databricks:main Mar 2, 2026
12 of 13 checks passed
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