The generic "Apache Solr server unavailable error" message suggests that there is an issue with the Apache Solr service, however there are numerous reasons other than the Solr server being down when this error is deiplayed. Although the detailed messages are logged via watchdog, the message displayed on the search page leads people to make judgements and post error reports to their administrators that their Solr server is down when there is some other application level error either on the module or server side that is causing the issue.

A couple of possible suggestions to consider, very open to others:

  • Change the message displayed to end users to something more generic
  • Optionally display the detailed error message via drupal_set_message() to users with the "administer search permissions"

Comments

cpliakas’s picture

Title: Change the wording of the generic "Apache Solr server unavailable error" message » Change the wording of the generic "Apache Solr server unavailable" error message

Fixing quotes in title.

pwolanin’s picture

I thought at some point we did have an option to show extra info to admins - maybe we should see if that got broken, or needs to be expanded.

cpliakas’s picture

@pwolanin, I definitely could have missed something, but the only thing I can see is the "Extra help messages for administrators" options which only adds notices to a page whenever Drupal changed content that needs reindexing.

pwolanin’s picture

Ok, I was thinking of that option, but not remembering what it does - we could maybe use that as the flag?

cpliakas’s picture

Yes, that seems very reasonable to me. As a side note, I still think we should reconsider the wording of the error message as well. I understand it can be overridden in settings.php, but since it is more of an end user-facing message, I'm not sure it is the best option to mention things like "Apache Solr" and "Server". Maybe saying something even more generic like "search is currently unavailable" or something similar, that would be OK if we coupled it with the detailed information using the technique you mentioned. If we agree that this is the right path forward and can also agree on a message, I don't mind taking a crack at a patch.

cpliakas’s picture

Assigned: Unassigned » cpliakas

This message is the bane of my existence. Assigning to myself.

cpliakas’s picture

Status: Active » Needs review
StatusFileSize
new1.98 KB

Proposed message change attached. The words "Apache Solr" probably shouldn't be displayed to end users.

pwolanin’s picture

Status: Needs review » Reviewed & tested by the community

Looks fine to me.

nick_vh’s picture

Version: 7.x-1.x-dev » 6.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Looking great, committed to 7.x-1.x with giving all credit to cpliakas! :)

cpliakas’s picture

Excellent. So when there is an inevitable spelling error, we know who did it :-P.

pwolanin’s picture

Status: Patch (to be ported) » Needs review
StatusFileSize
new2.09 KB
pwolanin’s picture

Status: Needs review » Fixed

committed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.