[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17427753#comment-17427753 ] Tim Allison commented on TIKA-3312: --- Y. Thank you [~jpl-jengelke] > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > Fix For: 2.1.0 > > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17427341#comment-17427341 ] John Engelke commented on TIKA-3312: Should this be closed? It looks like SLF4J and Log4J2 are now supported in v2.1.1, at least. See TIKA-3453. > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17331234#comment-17331234 ] Konstantin Gribov commented on TIKA-3312: - [~tallison], will do. I took a peek into it right now and found couple of things that I'd like to change in dependencies but it would required thoughtful and attentive approach not to break something ,) > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[
https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17331203#comment-17331203
]
Tim Allison commented on TIKA-3312:
---
[~grossws] speaking of TIKA-3149, if you have a chance, can you review this
migration to log4j 2 in {{main}}?
> Support Log4j2 jar in Tika-app.jar
> --
>
> Key: TIKA-3312
> URL: https://issues.apache.org/jira/browse/TIKA-3312
> Project: Tika
> Issue Type: Improvement
>Affects Versions: 1.22, 1.24.1
>Reporter: Charushila Nanekar
>Priority: Critical
>
> Latest version of Tika-app is using older version of Log4j jar which cause an
> issue when Tika-app get integrated with other 3rd Party Application which
> using latest log4j 2 jar.
> Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant
> improvements over its predecessor.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17329377#comment-17329377 ] Hudson commented on TIKA-3312: -- UNSTABLE: Integrated in Jenkins build Tika » tika-main-jdk8 #204 (See [https://ci-builds.apache.org/job/Tika/job/tika-main-jdk8/204/]) TIKA-3312 -- upgrade to log4j2 throughout (tallison: [https://github.com/apache/tika/commit/b35b1685990a0401e33de0697499e40cec680858]) * (edit) tika-batch/pom.xml * (add) tika-server/tika-server-client/src/test/resources/log4j2.properties * (add) tika-app/src/main/resources/log4j2_batch_process.properties * (edit) tika-parsers/tika-parsers-classic/tika-parsers-classic-package/pom.xml * (add) tika-app/src/test/resources/log4j2_batch_process_test.properties * (delete) tika-app/src/main/resources/log4j_batch_process.properties * (edit) tika-pipes/tika-emitters/tika-emitter-s3/pom.xml * (add) tika-app/src/main/resources/log4j2.properties * (add) tika-server/tika-server-core/src/test/resources/logging/log4j2_forked.xml * (edit) tika-fuzzing/pom.xml * (delete) tika-server/tika-server-client/src/test/resources/log4j.properties * (edit) tika-app/src/test/java/org/apache/tika/cli/TikaCLIBatchIntegrationTest.java * (edit) tika-pipes/tika-emitters/tika-emitter-solr/pom.xml * (edit) tika-parent/pom.xml * (edit) tika-parsers/tika-parsers-advanced/tika-age-recogniser/pom.xml * (delete) tika-server/tika-server-core/src/test/resources/log4j.properties * (add) tika-server/tika-server-core/src/test/resources/log4j2.properties * (edit) tika-core/pom.xml * (edit) tika-batch/src/test/java/org/apache/tika/batch/fs/BatchDriverTest.java * (delete) tika-server/tika-server-core/src/test/resources/logging/log4j_forked.xml * (edit) tika-parsers/tika-parsers-classic/tika-parsers-classic-modules/tika-parser-pdf-module/src/test/java/org/apache/tika/parser/pdf/PDFParserTest.java * (edit) tika-eval/tika-eval-app/src/main/java/org/apache/tika/eval/app/XMLErrorLogUpdater.java * (edit) tika-parsers/tika-parsers-classic/tika-parsers-classic-package/src/test/java/org/apache/tika/parser/pdf/PDFParserTest.java * (delete) tika-server/tika-server-classic/src/test/resources/log4j.properties * (edit) tika-eval/tika-eval-app/src/main/java/org/apache/tika/eval/app/io/XMLLogReader.java * (delete) tika-fuzzing/src/main/resources/log4j.properties * (edit) tika-eval/tika-eval-app/src/main/java/org/apache/tika/eval/app/io/XMLLogMsgHandler.java * (delete) tika-app/src/test/resources/log4j_batch_process_test.properties * (edit) tika-langdetect/tika-langdetect-lingo24/pom.xml * (edit) tika-server/tika-server-core/pom.xml * (add) tika-fuzzing/src/main/resources/log4j2.properties * (edit) tika-app/pom.xml * (edit) tika-eval/tika-eval-app/pom.xml * (edit) tika-pipes/tika-pipes-async/pom.xml * (edit) tika-server/tika-server-client/pom.xml * (delete) tika-app/src/main/resources/log4j.properties * (delete) tika-server/tika-server-client/src/main/resources/log4j.properties * (edit) tika-pipes/tika-fetch-iterators/tika-fetch-iterator-s3/pom.xml * (edit) tika-parsers/tika-parsers-classic/tika-parsers-classic-modules/tika-parser-pdf-module/pom.xml * (edit) tika-pipes/tika-fetchers/tika-fetcher-http/pom.xml * (add) tika-server/tika-server-classic/src/test/resources/log4j2.properties * (delete) tika-server/tika-server-classic/src/main/resources/log4j.properties * (edit) tika-app/src/main/java/org/apache/tika/cli/TikaCLI.java * (edit) tika-pipes/tika-fetchers/tika-fetcher-s3/pom.xml * (edit) tika-pipes/tika-pipes-integration-tests/pom.xml * (edit) tika-server/tika-server-core/src/test/java/org/apache/tika/server/core/IntegrationTestBase.java * (edit) tika-app/src/main/java/org/apache/tika/cli/BatchCommandLineBuilder.java * (edit) tika-server/tika-server-classic/bin/tika.in.sh * (add) tika-server/tika-server-client/src/main/resources/log4j2.properties * (edit) tika-langdetect/tika-langdetect-mitll-text/pom.xml * (edit) tika-parsers/tika-parsers-advanced/tika-dl/pom.xml * (add) tika-server/tika-server-classic/src/main/resources/log4j2.properties * (edit) tika-batch/src/main/java/org/apache/tika/batch/BatchProcessDriverCLI.java > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17298345#comment-17298345 ] Konstantin Gribov commented on TIKA-3312: - [~tallison], in that case I think it could safely go into tika-server-core since it's already end-user runnable application. What do you think about extracting a module with just a bunch of runtime deps and configs for all cli tools? > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17298189#comment-17298189 ] Tim Allison commented on TIKA-3312: --- Thank you, [~grossws]! The other question is where to put the logging implementation in tika-server...whether it is in tika-server-core (which comes with no parsers) or tika-server-classic...that comes with the parsers-classic parsers? I'm thinking tika-server-classic? > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17298099#comment-17298099 ] Konstantin Gribov commented on TIKA-3312: - I agree that we should upgrade to log4j2 or logback-classic as logging implementation. But I would advice against using tika-app as a library. [~cnanekar], could you tell us why you depend on it instead of tika-parsers/tika-batch etc? Than you could choose whichever logging impl you prefer with its configuration specific to your app. > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Commented] (TIKA-3312) Support Log4j2 jar in Tika-app.jar
[ https://issues.apache.org/jira/browse/TIKA-3312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17295230#comment-17295230 ] Tim Allison commented on TIKA-3312: --- I worry that this will break stuff in 1.x. Let's put this in 2.x. Fellow devs, any objections? > Support Log4j2 jar in Tika-app.jar > -- > > Key: TIKA-3312 > URL: https://issues.apache.org/jira/browse/TIKA-3312 > Project: Tika > Issue Type: Improvement >Affects Versions: 1.22, 1.24.1 >Reporter: Charushila Nanekar >Priority: Critical > > Latest version of Tika-app is using older version of Log4j jar which cause an > issue when Tika-app get integrated with other 3rd Party Application which > using latest log4j 2 jar. > Additionally, Apache Log4j 2 is an upgrade to Log4j that provides significant > improvements over its predecessor. -- This message was sent by Atlassian Jira (v8.3.4#803005)
