svn commit: r1333206 - /incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/engine/iterator/QueryIterAssign.java

[an...@apache.org]

Author: andy
Date: Wed May  2 21:07:03 2012
New Revision: 1333206

URL: http://svn.apache.org/viewvc?rev=1333206&view=rev
Log:
Fix: optimization may cause a sequence to push bindings through (extend).  c.f. (assign).

Modified:
    incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/engine/iterator/QueryIterAssign.java

Modified: incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/engine/iterator/QueryIterAssign.java
URL: http://svn.apache.org/viewvc/incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/engine/iterator/QueryIterAssign.java?rev=1333206&r1=1333205&r2=1333206&view=diff
==============================================================================
--- incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/engine/iterator/QueryIterAssign.java (original)
+++ incubator/jena/Jena2/ARQ/trunk/src/main/java/com/hp/hpl/jena/sparql/engine/iterator/QueryIterAssign.java Wed May  2 21:07:03 2012
@@ -74,7 +74,8 @@ public class QueryIterAssign extends Que
             // Check is already has a value; if so, must be sameValueAs
             if ( b.contains(v) )
             {
-                if ( mustBeNewVar )
+                // Optimization may linearize to push a stream through an (extend).  
+                if ( false && mustBeNewVar )
                     throw new QueryExecException("Already set: "+v) ;
                 
                 Node n2 = b.get(v) ;