Solr
  1. Solr
  2. SOLR-333

distributiondump.jsp gets the path to logs wrong

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0, 1.2
    • Fix Version/s: 1.3
    • Component/s: web gui
    • Labels:
      None
    • Environment:

      Tomcat 5.5

      Description

      I can't seem to get the Distribution Info page to work.

      I think it is because I have a custom path to solr. My path to the logs directory looks like this: /apps/search/solr/catalog/logs (The base url to solr ends up being http://domain.com/catalog) I have to create my path like this in order to have multiple instances of solr running in the same container.

      distributiondump.jsp is creating the path to the status file using: cwd + "/solr/logs/snapshot.current", which ends up being wrong.

      Would it be possible to make it use a rootdir variable like: rootdir + "/logs/snapshot.current"? If there is no way to determine the rootdir, perhaps it could be added to solrconfig.xml (or at least let it be overridden there.)

        Activity

        Hide
        Bill Au added a comment -

        I am going to update the JSP to behave the same way as Solr does in determining solr home:

        1) JNDI lookup
        2) system property solr.solr.home
        3) default to cwd

        Ken, so you will need to do either 1) or 2) to set Solr home.

        Show
        Bill Au added a comment - I am going to update the JSP to behave the same way as Solr does in determining solr home: 1) JNDI lookup 2) system property solr.solr.home 3) default to cwd Ken, so you will need to do either 1) or 2) to set Solr home.
        Hide
        Bill Au added a comment -

        It turns out to be a very simple change in the JSP as it already has obtained solr home from solr.core.Config. Attached in the patch. I will apply it tomorrow if no one objects.

        Show
        Bill Au added a comment - It turns out to be a very simple change in the JSP as it already has obtained solr home from solr.core.Config. Attached in the patch. I will apply it tomorrow if no one objects.
        Hide
        Bill Au added a comment -

        Patch committed. CHANGES.txt updated.

        Show
        Bill Au added a comment - Patch committed. CHANGES.txt updated.

          People

          • Assignee:
            Bill Au
            Reporter:
            Ken Prows
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development