Uploaded image for project: 'Blazegraph (by SYSTAP)'
  1. Blazegraph (by SYSTAP)
  2. BLZG-1347

Query time in NSS UI of "1min, 66sec, 523ms" (more than 66 seconds, but also quotes minutes)

    Details

    • Type: Bug
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: BLAZEGRAPH_RELEASE_1_5_2
    • Component/s: None
    • Labels:
      None

      Description

      Running a moderately slow query in firefox I got this somewhat surprising time.

        Activity

        jjc Jeremy Carroll created issue -
        jjc Jeremy Carroll made changes -
        Field Original Value New Value
        Attachment screenshot-1.png [ 10215 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v4 [ 15841 ] Trac Import v5 [ 16027 ]
        Hide
        bryanthompson bryanthompson added a comment -

        Jeremy, We need more information. There is nothing I can do with the data on this ticket.
        Bryan

        Show
        bryanthompson bryanthompson added a comment - Jeremy, We need more information. There is nothing I can do with the data on this ticket. Bryan
        Hide
        jjc Jeremy Carroll added a comment -

        The time is surprising because if quoting minutes the number of seconds should be less than 60.
        I think this is just a code inspection of the JS

        To repro, run a query that takes more than a minute

        Show
        jjc Jeremy Carroll added a comment - The time is surprising because if quoting minutes the number of seconds should be less than 60. I think this is just a code inspection of the JS To repro, run a query that takes more than a minute
        Hide
        bryanthompson bryanthompson added a comment -

        Ah! I missed that bit.

        Show
        bryanthompson bryanthompson added a comment - Ah! I missed that bit.
        bryanthompson bryanthompson made changes -
        Summary Query time in NSS UI of "1min, 66sec, 523ms" Query time in NSS UI of "1min, 66sec, 523ms" (more than 66 seconds, but also quotes minutes)
        bryanthompson bryanthompson made changes -
        Assignee bryanthompson [ bryanthompson ] igorkim [ igorkim ]
        Hide
        bryanthompson bryanthompson added a comment -

        Igor,

        Please look at this ticket. The workbench is in

        /bigdata-war/src/html.js/workbench.js
        

        Thanks,
        Bryan

        Show
        bryanthompson bryanthompson added a comment - Igor, Please look at this ticket. The workbench is in /bigdata-war/src/html.js/workbench.js Thanks, Bryan
        Hide
        igorkim igorkim added a comment - - edited

        already fixed by commit 1516c67c49e8fbf1b7956ee7b840211e7a6ad1bd Author: Peter Haase <ph@metaphacts.com> 2014-11-07 19:57:57

        tested using:
        Firefox 32.0.3, 34.0.5, 38.0.5
        Chrome 43.0.2357.81

        Show
        igorkim igorkim added a comment - - edited already fixed by commit 1516c67c49e8fbf1b7956ee7b840211e7a6ad1bd Author: Peter Haase <ph@metaphacts.com> 2014-11-07 19:57:57 tested using: Firefox 32.0.3, 34.0.5, 38.0.5 Chrome 43.0.2357.81
        Hide
        beebs Brad Bebee added a comment -

        Jeremy Carroll Can you confirm the version of the Blazegraph for which you see this error?

        Show
        beebs Brad Bebee added a comment - Jeremy Carroll Can you confirm the version of the Blazegraph for which you see this error?
        Hide
        jjc Jeremy Carroll added a comment -

        I believe it is:
        buildVersion=1.5.1

        Show
        jjc Jeremy Carroll added a comment - I believe it is: buildVersion=1.5.1
        Hide
        beebs Brad Bebee added a comment -

        igorkim Can you please test against the 1.5.1 release per Jeremy's comment?

        Show
        beebs Brad Bebee added a comment - igorkim Can you please test against the 1.5.1 release per Jeremy's comment?
        Hide
        igorkim igorkim added a comment - - edited

        Brad Bebee, Jeremy Carroll,

        We've checked 1.5.1 version (commit ee28e86a31462b2b2fc71796dae56cf39bec659c):
        workbench.js contains correct duration handling.
        github version is the same:
        sec = sec % 60;

        Show
        igorkim igorkim added a comment - - edited Brad Bebee , Jeremy Carroll , We've checked 1.5.1 version (commit ee28e86a31462b2b2fc71796dae56cf39bec659c): workbench.js contains correct duration handling. github version is the same: sec = sec % 60;
        beebs Brad Bebee made changes -
        Workflow Trac Import v5 [ 16027 ] Trac Import v6 [ 17480 ]
        bryanthompson bryanthompson made changes -
        Fix Version/s BLAZEGRAPH_RELEASE_1_5_2 [ 10164 ]
        bryanthompson bryanthompson made changes -
        Status Open [ 1 ] Accepted [ 10101 ]
        bryanthompson bryanthompson made changes -
        Status Accepted [ 10101 ] In Progress [ 3 ]
        bryanthompson bryanthompson made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        beebs Brad Bebee made changes -
        Fix Version/s BLAZEGRAPH_RELEASE_1_5_3 [ 10165 ]
        Fix Version/s BLAZEGRAPH_RELEASE_1_5_2 [ 10164 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v6 [ 17480 ] Trac Import v7 [ 19993 ]
        beebs Brad Bebee made changes -
        Status Resolved [ 5 ] In Review [ 10100 ]
        beebs Brad Bebee made changes -
        Resolution Done [ 10000 ]
        Status In Review [ 10100 ] Done [ 10000 ]
        bryanthompson bryanthompson made changes -
        Fix Version/s BLAZEGRAPH_RELEASE_1_5_2 [ 10164 ]
        Fix Version/s BLAZEGRAPH_RELEASE_1_5_3 [ 10165 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v7 [ 19993 ] Trac Import v8 [ 21427 ]

          People

          • Assignee:
            igorkim igorkim
            Reporter:
            jjc Jeremy Carroll
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: