[ 
https://issues.apache.org/jira/browse/TIKA-2439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16110844#comment-16110844
 ] 

Hudson commented on TIKA-2439:
------------------------------

SUCCESS: Integrated in Jenkins build Tika-trunk #1341 (See 
[https://builds.apache.org/job/Tika-trunk/1341/])
TIKA-2439 throwing IllegalStateException in (krichter722: 
[https://github.com/apache/tika/commit/57ee179862d6f1a15cea6745ca3d1c8e8e7ff800])
* (edit) 
tika-langdetect/src/main/java/org/apache/tika/langdetect/OptimaizeLangDetector.java


> Avoid NullPointerException in 
> org.apache.tika.langdetect.OptimaizeLangDetector if models haven't been loaded
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: TIKA-2439
>                 URL: https://issues.apache.org/jira/browse/TIKA-2439
>             Project: Tika
>          Issue Type: Improvement
>          Components: languageidentifier
>    Affects Versions: 1.17
>            Reporter: Karl Richter
>             Fix For: 1.17
>
>
> In an intuitive usage of
> {code:java}
> LanguageDetector languageDetector = 
> LanguageDetector.getDefaultLanguageDetector();
> List<LanguageResult> languageResults = 
> languageDetector.detectAll(someNonEmptyString);
> {code}
> `org.apache.tika.langdetect.OptimaizeLangDetector` might/will (?) be chosen 
> as default detector, but since there's no call to 
> `LanguageDetector.loadModels()` `LanguageDetector.detector` is `null` in 
> `detectAll` which causes an unhelpful error situation which one has to 
> investigate in the code. A simple check whether `detector` is `null` and 
> throwing an `IllegalStateException("models haven't been loaded yet (forgot to 
> call loadModels?")` would be much more intuitive.
> If that corresponds to the expected behaviour (it's my first week with Tika), 
> I can provide a patch or pull request.
> experienced with 1.16-75-g4455a6f08



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to