You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@livy.apache.org by "roczei (via GitHub)" <gi...@apache.org> on 2026/06/02 19:51:49 UTC
[PR] LIVY-1040: Migrate logging framework from Reload4j to Log4j2 [incubator-livy]
roczei opened a new pull request, #525:
URL: https://github.com/apache/incubator-livy/pull/525
## What changes were proposed in this pull request?
Replaced the reload4j dependency with the Log4j2 library across all modules.
Co-authored by Peter Toth and Gyorgy Gal:
- Added log4j-api, log4j-core, log4j-slf4j-impl and log4j-1.2-api (backward-compat bridge) to dependencies
- Removed ch.qos.reload4j:reload4j from all module poms
- Migrated all log4j.properties test config files to log4j2.properties using the new Log4j2 property-file syntax (rootLogger, appender.* keys)
## How was this patch tested?
It has been manually tested with the existing unit tests.
## Was this patch authored or co-authored using generative AI tooling?
No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@livy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [LIVY-1040] Migrate logging framework from Reload4j to Log4j2 [incubator-livy]
Posted by "roczei (via GitHub)" <gi...@apache.org>.
roczei commented on PR #525:
URL: https://github.com/apache/incubator-livy/pull/525#issuecomment-4629544444
Hi @ArnavBalyan, @peter-toth and @gyogal,
This PR is ready for code review. Could you please take a look at this when you have a moment? Any feedback you provide would be greatly appreciated! Thank you!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@livy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [LIVY-1040] Migrate logging framework from Reload4j to Log4j2 [livy]
Posted by "gyogal (via GitHub)" <gi...@apache.org>.
gyogal merged PR #525:
URL: https://github.com/apache/livy/pull/525
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@livy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [LIVY-1040] Migrate logging framework from Reload4j to Log4j2 [livy]
Posted by "gyogal (via GitHub)" <gi...@apache.org>.
gyogal commented on PR #525:
URL: https://github.com/apache/livy/pull/525#issuecomment-4732181316
Thank you for updating the PR @roczei ! I ran some tests with the updated `log4j2.properties.template` and the logs are looking good now:
```
26/06/16 17:08:21 INFO BatchSessionManager: Recovered 0 batch sessions. Next session id: 0
26/06/16 17:08:21 INFO InteractiveSessionManager: Recovered 0 interactive sessions. Next session id: 0
26/06/16 17:08:21 INFO InteractiveSessionManager: Heartbeat watchdog thread started.
```
I also checked with the Maven dependency plugin that reload4j had been completely removed from the dependencies, so the change looks good to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@livy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org