Details

      Description

      Workbench creates explore query which is way to complex, e.g. :

      select ?col1 ?col2 ?incoming (count(?star) as ?star) {
        bind (<http://collection.britishmuseum.org/id/person-institution/171533/activity/1/date> as ?explore ) .
        {
          bind (<<?explore ?col1 ?col2>> as ?sid) . 
          bind (false as ?incoming) . 
          optional {
            { ?sid ?sidP ?star } union { ?star ?sidP ?sid }
          }
        } union {
          bind (<<?col1 ?col2 ?explore>> as ?sid) .
          bind (true as ?incoming) . 
          optional {
            { ?sid ?sidP ?star } union { ?star ?sidP ?sid }
          }
        }
      }
      group by ?col1 ?col2 ?incoming
      

      This query runs on our server for many minutes, even so it fully consumes all 12 cores and many GB of ram. But in fact explored URI has only 3 incoming edges:

      select (count(*) as ?c) { 
        ?s ?p <http://collection.britishmuseum.org/id/person-institution/171533/activity/1/date>
      }
      

      Result:

      c
      3
      

      And 5 outgoing:

      select (count(*) as ?c) { 
        <http://collection.britishmuseum.org/id/person-institution/171533/activity/1/date> ?p ?o
      }
      

      Result:

      c
      5
      

        Issue Links

          Activity

          Hide
          michaelschmidt michaelschmidt added a comment -

          Possibly related to SPARQL semantics changes and the fact that the binding for ?explore is not projected into the UNION.

          Show
          michaelschmidt michaelschmidt added a comment - Possibly related to SPARQL semantics changes and the fact that the binding for ?explore is not projected into the UNION.
          Hide
          bryanthompson bryanthompson added a comment -

          Assigned back to Michael as it is related to the projection of ?explore into the UNION

          Show
          bryanthompson bryanthompson added a comment - Assigned back to Michael as it is related to the projection of ?explore into the UNION
          Hide
          bryanthompson bryanthompson added a comment -

          We might want to push this out to the 1.5.2 maintenance branch as well since it impacts the GUI

          Show
          bryanthompson bryanthompson added a comment - We might want to push this out to the 1.5.2 maintenance branch as well since it impacts the GUI
          Hide
          michaelschmidt michaelschmidt added a comment - - edited

          Solved performance issue by modification of the queries submitted by the workbench: the queries now execute BIND in inner scopes where they are visible.

          Note that this problem was not introduced in 1.5.2 but showed up before already.

          Please merge down, see branch BLZG-1443.

          Show
          michaelschmidt michaelschmidt added a comment - - edited Solved performance issue by modification of the queries submitted by the workbench: the queries now execute BIND in inner scopes where they are visible. Note that this problem was not introduced in 1.5.2 but showed up before already. Please merge down, see branch BLZG-1443 .
          Hide
          beebs Brad Bebee added a comment -

          Will merge done today. Do you also want to push an open source release on SF with this?

          Show
          beebs Brad Bebee added a comment - Will merge done today. Do you also want to push an open source release on SF with this?
          Hide
          beebs Brad Bebee added a comment -

          Per thompsonbry Merge into maintenance branch and then push branch to SF.

          Show
          beebs Brad Bebee added a comment - Per thompsonbry Merge into maintenance branch and then push branch to SF.
          Hide
          beebs Brad Bebee added a comment -

          Merged down after clean CI. Pushed into BLAZEGRAPH_RELEASE_1_5_2_MAINTENANCE branch in Git and Sourceforge.

          Show
          beebs Brad Bebee added a comment - Merged down after clean CI. Pushed into BLAZEGRAPH_RELEASE_1_5_2_MAINTENANCE branch in Git and Sourceforge.

            People

            • Assignee:
              beebs Brad Bebee
              Reporter:
              akozlov Artem Kozlov
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: