[jira] [Commented] (TIKA-3665) Clean up default dynamic loading in ServiceLoader

["Tim Allison (Jira)" <ji...@apache.org>]

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

Tim Allison commented on TIKA-3665:
-----------------------------------

I think I made some progress on this, but there's a bunch more that should be done. No need to hold up 2.3.0 for this, though.

> Clean up default dynamic loading in ServiceLoader
> -------------------------------------------------
>
>                 Key: TIKA-3665
>                 URL: https://issues.apache.org/jira/browse/TIKA-3665
>             Project: Tika
>          Issue Type: Task
>            Reporter: Tim Allison
>            Priority: Major
>
> Over the years, it looks like we've grown some weird behavior around dynamic loading of resources.
> My understanding is that we only want dynamic loading in OSGi environments.
> However, the default behavior of whether or not the serviceloaders are dynamic is weird: TikaConfig() has default dynamic=true, TikaConfig(ClassLoader) dynamic=false, TikaConfig(InputStream|Path) (if service-loader element is not set to true) dynamic=false.
> Further, it looks like some components are dynamic=true by default where as others are dynamic=false.  I suspect no one has noticed that the DefaultEncodingDetector has default = false.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)