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

      select $a
      { VALUES $a { 1 1  1 1 } }
      

      gives 16 answers, rather than 4

        Activity

        jjc Jeremy Carroll created issue -
        bryanthompson bryanthompson made changes -
        Field Original Value New Value
        Assignee bryanthompson [ bryanthompson ] michaelschmidt [ michaelschmidt ]
        Hide
        michaelschmidt michaelschmidt added a comment -

        This might be a duplicate of BLZG-1256 and will be covered as part of the refactoring.

        Show
        michaelschmidt michaelschmidt added a comment - This might be a duplicate of BLZG-1256 and will be covered as part of the refactoring.
        Hide
        michaelschmidt michaelschmidt added a comment -

        Fixed in branch bindings-refactor, added ticket as a test case. Just re-running govtrack to get hints on performance regressions, once this succeeds the branch will be merged into master.

        The fix was part of a major refactoring. The prior optimizers ASTSimpleBindingsOptimizer, ASTValuesOptimizer, and ASTBindingsAssigner were retired, and custom code for extracting static bindings (outside of optimizers) was eliminated. All cases were generalized in the ASTStaticBindingsOptimizer, which analyzes the query to detect statically derivable bindings introduced via constructs (such as BIND/VALUES, FILTER with sameTerm, IN or term equality over URIs). These bindings are pulled out to the top-level clause where valid (according to the bottom up semantics) and inlines them at various positions. The refactoring is backed by a broad set of test cases, including both SPARQL level and AST level tests.

        Show
        michaelschmidt michaelschmidt added a comment - Fixed in branch bindings-refactor, added ticket as a test case. Just re-running govtrack to get hints on performance regressions, once this succeeds the branch will be merged into master. The fix was part of a major refactoring. The prior optimizers ASTSimpleBindingsOptimizer, ASTValuesOptimizer, and ASTBindingsAssigner were retired, and custom code for extracting static bindings (outside of optimizers) was eliminated. All cases were generalized in the ASTStaticBindingsOptimizer, which analyzes the query to detect statically derivable bindings introduced via constructs (such as BIND/VALUES, FILTER with sameTerm, IN or term equality over URIs). These bindings are pulled out to the top-level clause where valid (according to the bottom up semantics) and inlines them at various positions. The refactoring is backed by a broad set of test cases, including both SPARQL level and AST level tests.
        michaelschmidt michaelschmidt made changes -
        Status Open [ 1 ] Accepted [ 10101 ]
        michaelschmidt michaelschmidt made changes -
        Status Accepted [ 10101 ] In Progress [ 3 ]
        michaelschmidt michaelschmidt made changes -
        Status In Progress [ 3 ] Done [ 10000 ]
        michaelschmidt michaelschmidt made changes -
        Fix Version/s BLAZEGRAPH_RELEASE_1_5_2 [ 10164 ]
        michaelschmidt michaelschmidt made changes -
        Status Done [ 10000 ] In Review [ 10100 ]
        michaelschmidt michaelschmidt made changes -
        Status In Review [ 10100 ] Resolved [ 5 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v2 [ 13135 ] Trac Import v3 [ 13524 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v3 [ 13524 ] Trac Import v4 [ 14854 ]
        beebs Brad Bebee made changes -
        Status Resolved [ 5 ] In Review [ 10100 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v4 [ 14854 ] Trac Import v5 [ 16078 ]
        beebs Brad Bebee made changes -
        Resolution Done [ 10000 ]
        Status In Review [ 10100 ] Done [ 10000 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v5 [ 16078 ] Trac Import v6 [ 18380 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v6 [ 18380 ] Trac Import v7 [ 19782 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v7 [ 19782 ] Trac Import v8 [ 21409 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: