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

Duplicate LET expression leading to NotMaterializedException

    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

      Consider data

      <http://s> <http://p> "val” .
      

      and query

      SELECT * WHERE
      {
        ?s ?p ?o .
        LET (?tmp := ?o) .
        LET(?tmp2 := sameTerm(?tmp,"val2") ) .
      }
      

      The query throws a NonMaterializedException.

        Activity

        michaelschmidt michaelschmidt created issue -
        Hide
        michaelschmidt michaelschmidt added a comment -

        There was a problem with the translation pattern of the first LET into a BOp, adding variable ?tmp to the doneSet. Fixed in branch "not-materialized-exception", pull request issued.

        Show
        michaelschmidt michaelschmidt added a comment - There was a problem with the translation pattern of the first LET into a BOp, adding variable ?tmp to the doneSet. Fixed in branch "not-materialized-exception", pull request issued.
        Hide
        bryanthompson bryanthompson added a comment -

        One line fix to AST2BOpUtility.addResolvedAssignment() per https://github.com/SYSTAP/bigdata/pull/76.

                left = addVariableUnificationOp(
                         left, assignmentNode.getVar(), freshVar, ctx);        
                
        -       /**
        -        * The assignment node var is definitely resolved now
        -        */
        -       doneSet.add(assignmentNode.getVar());
        -       
                return left;       
            }
        
        Show
        bryanthompson bryanthompson added a comment - One line fix to AST2BOpUtility.addResolvedAssignment() per https://github.com/SYSTAP/bigdata/pull/76 . left = addVariableUnificationOp( left, assignmentNode.getVar(), freshVar, ctx); - /** - * The assignment node var is definitely resolved now - */ - doneSet.add(assignmentNode.getVar()); - return left; }
        michaelschmidt michaelschmidt made changes -
        Field Original Value New Value
        Status Open [ 1 ] Accepted [ 10101 ]
        michaelschmidt michaelschmidt made changes -
        Status Accepted [ 10101 ] In Progress [ 3 ]
        michaelschmidt michaelschmidt made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        michaelschmidt michaelschmidt made changes -
        Status Resolved [ 5 ] In Review [ 10100 ]
        michaelschmidt michaelschmidt made changes -
        Status In Review [ 10100 ] Done [ 10000 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v4 [ 15825 ] Trac Import v5 [ 16070 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v5 [ 16070 ] Trac Import v6 [ 18387 ]
        michaelschmidt michaelschmidt made changes -
        Fix Version/s BLAZEGRAPH_RELEASE_1_5_2 [ 10164 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v6 [ 18387 ] Trac Import v7 [ 19791 ]
        beebs Brad Bebee made changes -
        Workflow Trac Import v7 [ 19791 ] Trac Import v8 [ 21422 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: