Solr
  1. Solr
  2. SOLR-2482

DataImportHandler; reload-config; response in case of failure & further requests

    Details

      Description

      Reloading while the config-file is valid is completely fine, but if the config is broken - the Response is plain HTML, containing the full stacktrace (see attachment). further requests contain a status Element with DataImportHandler started. Not Initialized. No commands can be run, but respond with a HTTP-Status 200 OK :/

      Would be nice, if:

      • the response in case of error could also be xml formatted
      • contain the exception message (in my case The end-tag for element type "entity" must end with a '>' delimiter.) in a seperate field
      • use a better/correct http-status for the latter mentioned requests, i would suggest 503 Service Unavailable

      So we are able to display to error-message to the user, while the config gets broken - and for the further requests we could rely on the http-status and have no need to check the content of the xml-response.

      1. reload-config-error.html
        10 kB
        Stefan Matheis (steffkes)

        Issue Links

          Activity

          Uwe Schindler made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Gavin made changes -
          Link This issue is depended upon by SOLR-2399 [ SOLR-2399 ]
          Gavin made changes -
          Link This issue blocks SOLR-2399 [ SOLR-2399 ]
          James Dyer made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee James Dyer [ jdyer ]
          Fix Version/s 4.0 [ 12322455 ]
          Resolution Fixed [ 1 ]
          James Dyer made changes -
          Link This issue relates to SOLR-2115 [ SOLR-2115 ]
          Stefan Matheis (steffkes) made changes -
          Link This issue blocks SOLR-3162 [ SOLR-3162 ]
          Stefan Matheis (steffkes) made changes -
          Link This issue blocks SOLR-2399 [ SOLR-2399 ]
          Stefan Matheis (steffkes) made changes -
          Field Original Value New Value
          Attachment reload-config-error.html [ 12477834 ]
          Stefan Matheis (steffkes) created issue -

            People

            • Assignee:
              James Dyer
              Reporter:
              Stefan Matheis (steffkes)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development