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

        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.
        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;

          People

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

            Dates

            • Created:
              Updated:
              Resolved: