Solr
  1. Solr
  2. SOLR-4456

Admin UI: Displays dashboard even if Solr is down

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.1
    • Fix Version/s: 4.4, 6.0
    • Component/s: web gui
    • Labels:
      None

      Description

      1. Run Solr and bruing up the Admin dashboard
      2. Stop Solr
      3. Click around the Admin GUI. It apparently works, but displays a spinning wheel for most panels
      4. Click on "Dashboard". An old cached dashboard is displayed

      What should happen is that once connection to Solr is lost, the whole Admin UI displays a large red box "CONNECTION LOST" or something

      1. SOLR-4456.patch
        3 kB
        Stefan Matheis (steffkes)
      2. SOLR-4456.patch
        3 kB
        Stefan Matheis (steffkes)
      3. SOLR-4456.patch
        0.8 kB
        Stefan Matheis (steffkes)

        Activity

        Hide
        Jan Høydahl added a comment -

        Have you seen this, Stefan Matheis (steffkes)? Is it related to caching in any way, like SOLR-4311?

        Show
        Jan Høydahl added a comment - Have you seen this, Stefan Matheis (steffkes) ? Is it related to caching in any way, like SOLR-4311 ?
        Hide
        Stefan Matheis (steffkes) added a comment -

        Hey Jan Høydahl, yes already saw this, but had no clue until yesterday if there is a way to detect that properly ;o

        That's not related to the caching issue .. that's another kind of caching :/ It's still in the Browser DOM (Document Object Model), that's why you can see it w/o having a server-connection. I'll try to bring up a hackish patch that we can check if the detection works the way it should .. and if it does, i'm going to put some layout over it and we're done, hopefully *g

        Show
        Stefan Matheis (steffkes) added a comment - Hey Jan Høydahl , yes already saw this, but had no clue until yesterday if there is a way to detect that properly ;o That's not related to the caching issue .. that's another kind of caching :/ It's still in the Browser DOM (Document Object Model), that's why you can see it w/o having a server-connection. I'll try to bring up a hackish patch that we can check if the detection works the way it should .. and if it does, i'm going to put some layout over it and we're done, hopefully *g
        Hide
        Stefan Matheis (steffkes) added a comment -

        Jan Høydahl would you mind trying this? I'm mainly interested in the fact if the 'offline'-detection works properly – having said that, if you're using the UI, shutting Solr down and try to continue, if a dark overlay containing a white box which says "Connection lost ..." appears on your screen?

        In addition to that, my idea would be:

        • Requesting a ressource (f.e. /admin/system or sth like that) to discover if we're back online
        • If so, displaying another note saying "Reload the Page to continue"

        What i would like to do, is: resume the UI where it stopped – but i'm not sure if that will work out. For that we have to track every single request and try to re-execute it again ..

        Show
        Stefan Matheis (steffkes) added a comment - Jan Høydahl would you mind trying this? I'm mainly interested in the fact if the 'offline'-detection works properly – having said that, if you're using the UI, shutting Solr down and try to continue, if a dark overlay containing a white box which says "Connection lost ..." appears on your screen? In addition to that, my idea would be: Requesting a ressource (f.e. /admin/system or sth like that) to discover if we're back online If so, displaying another note saying "Reload the Page to continue" What i would like to do, is: resume the UI where it stopped – but i'm not sure if that will work out. For that we have to track every single request and try to re-execute it again ..
        Hide
        Jan Høydahl added a comment -

        Yep, it works well. I can see the white box in Chrome on Mac.

        Don't think it's important to resume the UI where you left. But perhaps the white alert box could have a clickable link "retry" which simply reloads the page?

        Show
        Jan Høydahl added a comment - Yep, it works well. I can see the white box in Chrome on Mac. Don't think it's important to resume the UI where you left. But perhaps the white alert box could have a clickable link "retry" which simply reloads the page?
        Hide
        Stefan Matheis (steffkes) added a comment -

        Updated Patch. Now, if the Layer appears, the Browser will be checking in the background if the server is available again. (the background check uses a exponential function to avoid hammering the server)

        When that's the case, all the bad-red things will change into friendly-green ones, telling you that the instance is available again und that you should reload the page.

        While playing around with it, i though about automatically reloading the page .. but i'm not really sure if that's clear what happend from a users perspective? If you're not sitting in front of the browser (especially this tab) and you view it again and the layer is done w/o you doing anything?

        Show
        Stefan Matheis (steffkes) added a comment - Updated Patch. Now, if the Layer appears, the Browser will be checking in the background if the server is available again. (the background check uses a exponential function to avoid hammering the server) When that's the case, all the bad-red things will change into friendly-green ones, telling you that the instance is available again und that you should reload the page. While playing around with it, i though about automatically reloading the page .. but i'm not really sure if that's clear what happend from a users perspective? If you're not sitting in front of the browser (especially this tab) and you view it again and the layer is done w/o you doing anything?
        Hide
        Stefan Matheis (steffkes) added a comment -

        committed the current state in
        trunk r1494765
        branch_4x r1494768

        if there a suggestions for tweaking take, please open a new ticket for that

        Show
        Stefan Matheis (steffkes) added a comment - committed the current state in trunk r1494765 branch_4x r1494768 if there a suggestions for tweaking take, please open a new ticket for that
        Hide
        Steve Rowe added a comment -

        Bulk close resolved 4.4 issues

        Show
        Steve Rowe added a comment - Bulk close resolved 4.4 issues

          People

          • Assignee:
            Stefan Matheis (steffkes)
            Reporter:
            Jan Høydahl
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development