Details

      Description

      With a dataset with multiple graphs the above query should report the names of all the named graphs, in fact it returns a single row, with ?g unbound.

      The ASTGraphGroupOptimizer is responsible for these matters.

      See [1] for discussion.

      See [2,3] for related tickets.

      [1] http://lists.w3.org/Archives/Public/public-sparql-dev/2013JulSep/0016.html
      [2] https://sourceforge.net/apps/trac/bigdata/ticket/429 (Optimization for GRAPH uri {} and GRAPH ?foo {})
      [3] https://sourceforge.net/apps/trac/bigdata/ticket/792 (GRAPH ?g { FILTER NOT EXISTS { ?s ?p ?o } } not respecting ?g)

        Activity

        Hide
        jeremycarroll jeremycarroll added a comment -

        Jeremy to create test case and commit, then reassign to bryan

        Show
        jeremycarroll jeremycarroll added a comment - Jeremy to create test case and commit, then reassign to bryan
        Hide
        bryanthompson bryanthompson added a comment -

        Added a link to a related ticket.

        Show
        bryanthompson bryanthompson added a comment - Added a link to a related ticket.
        Hide
        jeremycarroll jeremycarroll added a comment -

        The tests are in commit 7307
        and run from bigdata-rdf/src/test/com/bigdata/rdf/sparql/ast/eval/TestGraphEmptyPattern709_429.java
        This is not linked to a wider test suite

        Show
        jeremycarroll jeremycarroll added a comment - The tests are in commit 7307 and run from bigdata-rdf/src/test/com/bigdata/rdf/sparql/ast/eval/TestGraphEmptyPattern709_429.java This is not linked to a wider test suite
        Hide
        michaelschmidt michaelschmidt added a comment -

        Added special case handling that was foreseen in ASTGraphGroupOptimizer but not yet implemented. The idea is to use the distinct term scan pattern to evaluate the construct, i.e. we rewrite

        GRAPH ?g { }

        into

        GRAPH ?g { DistinctTermScan(?_s, ?_p, ?_o, ?g) }

        Issued pull request, see branch ticket709.

        Show
        michaelschmidt michaelschmidt added a comment - Added special case handling that was foreseen in ASTGraphGroupOptimizer but not yet implemented. The idea is to use the distinct term scan pattern to evaluate the construct, i.e. we rewrite GRAPH ?g { } into GRAPH ?g { DistinctTermScan(?_s, ?_p, ?_o, ?g) } Issued pull request, see branch ticket709.
        Hide
        michaelschmidt michaelschmidt added a comment -

        Also note that I did not implement the edge case "GRAPH <uri> { }" as part of this patch -> it is covered separately in bug http://trac.bigdata.com/ticket/429.

        Show
        michaelschmidt michaelschmidt added a comment - Also note that I did not implement the edge case "GRAPH <uri> { }" as part of this patch -> it is covered separately in bug http://trac.bigdata.com/ticket/429 .
        Hide
        michaelschmidt michaelschmidt added a comment -

        Merged into master, closing issue.

        Show
        michaelschmidt michaelschmidt added a comment - Merged into master, closing issue.

          People

          • Assignee:
            bryanthompson bryanthompson
            Reporter:
            jeremycarroll jeremycarroll
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: