You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2013/06/12 21:10:34 UTC

svn commit: r1492379 - /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/engine/main/OpExecutor.java

Author: andy
Date: Wed Jun 12 19:10:34 2013
New Revision: 1492379

URL: http://svn.apache.org/r1492379
Log:
Correctly print warning for attempt to use the union graph name.

Modified:
    jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/engine/main/OpExecutor.java

Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/engine/main/OpExecutor.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/engine/main/OpExecutor.java?rev=1492379&r1=1492378&r2=1492379&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/engine/main/OpExecutor.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/engine/main/OpExecutor.java Wed Jun 12 19:10:34 2013
@@ -159,16 +159,10 @@ public class OpExecutor
             return execute(subOp, input, cxt2) ;
         }
 
-        if (gn == Quad.unionGraph)
+        if ( Quad.isUnionGraph(gn) )
             Log.warn(this, "Not implemented yet: union default graph in general OpExecutor") ;
 
-        if (true)
-            return null ;
-
-        /*
-         * Bad if ( gn == Quad.tripleInQuad ) {}
-         */
-
+        // Bad news -- if ( Lib.equals(gn, Quad.tripleInQuad) ) {}
         return null ;
     }