[
https://issues.apache.org/jira/browse/SOLR-14197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17056993#comment-17056993
]
ASF subversion and git services commented on SOLR-14197:
--------------------------------------------------------
Commit 732348ec7f9c6b6f7bf9d539a40e50d16198cccc in lucene-solr's branch
refs/heads/master from David Smiley
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=732348e ]
SOLR-14197: SolrResourceLoader refactorings to reduce API
* Remove SRL.listConfigDir (unused)
* Remove SRL.getDataDir
* Remove SRL.getCoreName
* Remove SRL.getCoreProperties
XmlConfigFile needs to be passed in the substitutableProperties
IndexSchema needs to be passed in the substitutableProperties
Remove redundant Properties from CoreContainer constructors
* Remove SRL.newAdminHandlerInstance (unused)
* Remove SRL.openSchema and openConfig
* Avoid SRL.getConfigDir
Also harmonized similar initialization logic between DIH Tika processor &
ExtractingRequestHandler.
* Ensure SRL.addToClassLoader and reloadLuceneSPI are called at most once
Don't auto-load "lib" in constructor; wrong place for this logic.
* Avoid SRL.getInstancePath
Added SolrCore.getInstancePath instead
Use CoreContainer.getSolrHome instead
NodeConfig should track solrHome separate from SolrResourceLoader
* Simplify some SolrCore constructors
* Move locateSolrHome to new SolrPaths
* Move "User Files" stuff to SolrPaths
> Reduce API of SolrResourceLoader
> --------------------------------
>
> Key: SOLR-14197
> URL: https://issues.apache.org/jira/browse/SOLR-14197
> Project: Solr
> Issue Type: Improvement
> Reporter: David Smiley
> Assignee: David Smiley
> Priority: Major
> Time Spent: 3h 40m
> Remaining Estimate: 0h
>
> SolrResourceLoader must exist as an implementation of Lucene ResourceLoader.
> Over time it has grown a lot more methods that are beyond the scope of doing
> just that. I think SRL should be simplified to get back to its core
> necessity, which hopefully allows us to make other changes (either in SRL or
> nearby) with less entanglement. Some API creep:
> * listConfigDir _(not used)_
> * newAdminHandlerInstance _(not used)_
> * getDataDir
> * getCoreName _(only used for logging)_
> * getCoreProperties
> * openSchema and configSchema _(not needed; barely used)_
> Some methods ought to go to a new subclass of SolrResourceLoader that is
> specifically for a standalone-mode core:
> * resourceLocation
> * getConfigDir
> * getInstancePath
> * persistConfLocally
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
