lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jan Høydahl (JIRA) <>
Subject [jira] [Commented] (SOLR-7727) bin/solr script incorrectly requires solr.xml in $SOLR_HOME
Date Sun, 28 Jun 2015 21:12:05 GMT


Jan Høydahl commented on SOLR-7727:

Thinking more about this, an alternative solution is to change {{SolrDispatchFilter}} to look
for {{solr.xml}} in ZK by default in cloud mode, and fallback to {{SOLR_HOME}} for back compat,
getting rid of {{-Dsolr.solrxml.location}}. Do you see any problems with such a solution?

Aside: Currently, if you start Solr in non-cloud mode with solrxml.location specified, e.g.
{{bin/solr start -Dsolr.solrxml.location=zookeeper}}, Jetty starts up but in a bad state,
instead of failing fast, but no error is given in the start script. Trying to access Solr
gives 500 error with nullpointer. This would also be fixed with the code-change above, since
loading solr.xml from ZK would only be attempted when in ZK mode.

> bin/solr script incorrectly requires solr.xml in $SOLR_HOME
> -----------------------------------------------------------
>                 Key: SOLR-7727
>                 URL:
>             Project: Solr
>          Issue Type: Bug
>          Components: scripts and tools
>    Affects Versions: 5.2.1
>            Reporter: Crawdaddy
>            Assignee: Jan Høydahl
>             Fix For: 5.3, Trunk
>         Attachments: SOLR-7727.patch
> The bin/solr script currently requires solr.xml to exist in $SOLR_HOME, going against
SOLR-4718 which allows it to be hosted in Zookeeper.
> Line 1219:
> if [ ! -e "$SOLR_HOME/solr.xml" ]; then
>   echo -e "\nSolr home directory $SOLR_HOME must contain a solr.xml file!\n"
>   exit 1
> fi
> Commenting this out, and specifying the following in one's, allows solr.xml
to once again be hosted in ZK:
> SOLR_OPTS="$SOLR_OPTS -Dsolr.solrxml.location=zookeeper"

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message