[
https://issues.apache.org/jira/browse/OPENNLP-1476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17696330#comment-17696330
]
ASF GitHub Bot commented on OPENNLP-1476:
-----------------------------------------
mawiesne commented on PR #512:
URL: https://github.com/apache/opennlp/pull/512#issuecomment-1454134469
@kinow / @rzo1 / @jzonthemtn Please see my "Notes" comment in the PR
description.
If possible on your side, please report back whether faster built-times are
reproducible in your dev setup.
With it we could estimate a performance increase for certain scenarios.
> Modernize DictionaryEntryPersistor to create XMLReader via
> javax.xml.parsers.SAXParserFactory
> ---------------------------------------------------------------------------------------------
>
> Key: OPENNLP-1476
> URL: https://issues.apache.org/jira/browse/OPENNLP-1476
> Project: OpenNLP
> Issue Type: Improvement
> Affects Versions: 2.1.0, 2.1.1
> Reporter: Martin Wiesner
> Assignee: Martin Wiesner
> Priority: Minor
> Fix For: 2.1.2
>
>
> Atm, {{org.xml.sax.helpers.XMLReaderFactory}} is used in
> {{DictionaryEntryPersistor}}. However, this the use of XMLReaderFactory is
> deprecated since Java 9, that is:
> {code:java}
> Deprecated
> It is recommended to use javax.xml.parsers.SAXParserFactory instead.
> {code}
> Aim(s):
> * Switch to {{SAXParserFactory}} to obtain instances of {{XMLReader}} to
> modern way.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
