You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pr@jena.apache.org by GitBox <gi...@apache.org> on 2021/03/04 09:25:10 UTC
[GitHub] [jena] Claudenw commented on pull request #946: Upgrade Jena JDBC to Log4j2 (JENA-1867)
Claudenw commented on pull request #946:
URL: https://github.com/apache/jena/pull/946#issuecomment-790466769
Since we seem to ship slf4j why not use it instead?
On Wed, Mar 3, 2021 at 7:47 PM Rob Vesse <no...@github.com> wrote:
> Drops Log4j 1.x from Jena JDBC in favour of Log4j 2.x, deprecates the
> functionality that cannot be replicated with Log4j 2.x
>
> NB - I blindly hit auto-format on the JenaDriver class not realising how
> much it would reformat. I will clean that up to just the actual changes for
> this PR tomorrow
> ------------------------------
> You can view, comment on, or merge this pull request online at:
>
> https://github.com/apache/jena/pull/946
> Commit Summary
>
> - Upgrade Jena JDBC to Log4j2 (JENA-1867)
>
> File Changes
>
> - *M* jena-jdbc/jena-jdbc-core/pom.xml
> <https://github.com/apache/jena/pull/946/files#diff-a9d80e55ff903a3573eedef1e83ae378043ff439edb7362883bd8116735b173e>
> (19)
> - *M*
> jena-jdbc/jena-jdbc-core/src/main/java/org/apache/jena/jdbc/JenaDriver.java
> <https://github.com/apache/jena/pull/946/files#diff-59180b3cbf1a6c2c56f2245335e293703ebe71aac568c6804882b438ad65c807>
> (294)
> - *M* jena-jdbc/jena-jdbc-driver-bundle/pom.xml
> <https://github.com/apache/jena/pull/946/files#diff-b67ab51dc4c33d472ce88fee0a9f12a93bff5d3535cdd44f84b3fb8447554036>
> (7)
> - *M*
> jena-jdbc/jena-jdbc-driver-mem/src/test/java/org/apache/jena/jdbc/mem/TS_JdbcDriverMem.java
> <https://github.com/apache/jena/pull/946/files#diff-a69da737f9ae6e71f19048e82152f150b24a5de4f8ff717774076c55e97aa4d5>
> (1)
> - *D*
> jena-jdbc/jena-jdbc-driver-mem/src/test/java/org/apache/jena/jdbc/mem/TestMemDriverWithLogging.java
> <https://github.com/apache/jena/pull/946/files#diff-b0e7b0efb801e16348c7411ce1d2d4ae506895b4c5eb1f2cd66891eac5f48106>
> (63)
> - *D*
> jena-jdbc/jena-jdbc-driver-mem/src/test/resources/log4j.properties
> <https://github.com/apache/jena/pull/946/files#diff-e60ff3d75d099b9997f21e1a03a81c1f4b3faa200536441b80158ec1ffef36dd>
> (29)
> - *M* jena-jdbc/pom.xml
> <https://github.com/apache/jena/pull/946/files#diff-c1eb65bdbe8aaf2c01dec6824e9b2d3a20b0ddf33e8cc2c61e98da89bf55163c>
> (15)
>
> Patch Links:
>
> - https://github.com/apache/jena/pull/946.patch
> - https://github.com/apache/jena/pull/946.diff
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/jena/pull/946>, or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AASTVHXZBKYA6XVAKRQCXG3TB2G5XANCNFSM4YR3UILQ>
> .
>
--
I like: Like Like - The likeliest place on the web
<http://like-like.xenei.com>
LinkedIn: http://www.linkedin.com/in/claudewarren
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org