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/28 12:56:33 UTC

svn commit: r1497729 - in /jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets: SPARQL_REST.java SPARQL_REST_R.java

Author: andy
Date: Fri Jun 28 10:56:32 2013
New Revision: 1497729

URL: http://svn.apache.org/r1497729
Log:
Use plain RDF/XML for GSP GET.

Modified:
    jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST.java
    jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST_R.java

Modified: jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST.java?rev=1497729&r1=1497728&r2=1497729&view=diff
==============================================================================
--- jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST.java (original)
+++ jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST.java Fri Jun 28 10:56:32 2013
@@ -330,19 +330,6 @@ public abstract class SPARQL_REST extend
         {
             Target dest = determineTarget(action) ;
             FusekiLib.addDataInto(data, dest.dsg, dest.graphName) ;
-            
-//            Graph g = dest.graph() ;
-//            
-//            if (g == null)
-//            {
-//                if (dest.isDefault) errorOccurred("Dataset does not have a default graph") ;
-//                log.info(format("[%d] Creating in-memory graph for <%s>", action.id, dest.graphName)) ;
-//                // Not default graph.
-//                // Not an autocreate dataset - create something.
-//                g = GraphFactory.createDefaultGraph() ;
-//                dest.dsg.addGraph(dest.graphName, g) ;
-//            }
-//            GraphUtil.addInto(g, data) ;
         } catch (RuntimeException ex)
         {
             // If anything went wrong, try to backout.

Modified: jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST_R.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST_R.java?rev=1497729&r1=1497728&r2=1497729&view=diff
==============================================================================
--- jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST_R.java (original)
+++ jena/trunk/jena-fuseki/src/main/java/org/apache/jena/fuseki/servlets/SPARQL_REST_R.java Fri Jun 28 10:56:32 2013
@@ -27,9 +27,7 @@ import javax.servlet.ServletOutputStream
 import org.apache.jena.atlas.web.MediaType ;
 import org.apache.jena.atlas.web.TypedOutputStream ;
 import org.apache.jena.fuseki.HttpNames ;
-import org.apache.jena.riot.Lang ;
-import org.apache.jena.riot.RDFDataMgr ;
-import org.apache.jena.riot.WebContent ;
+import org.apache.jena.riot.* ;
 
 import com.hp.hpl.jena.graph.Graph ;
 import com.hp.hpl.jena.rdf.model.Model ;
@@ -64,7 +62,7 @@ public class SPARQL_REST_R extends SPARQ
 
         if ( action.verbose )
             log.info(format("[%d]   Get: Content-Type=%s, Charset=%s => %s", 
-                                  action.id, mediaType.getContentType(), mediaType.getCharset(), lang.getName())) ;
+                            action.id, mediaType.getContentType(), mediaType.getCharset(), lang.getName())) ;
 
         action.beginRead() ;
 
@@ -81,7 +79,11 @@ public class SPARQL_REST_R extends SPARQ
             action.response.setContentType(ct) ;
             Graph g = target.graph() ;
             Model model = ModelFactory.createModelForGraph(g) ;
-            RDFDataMgr.write(out, model, lang) ;
+            
+            //Special case RDF/XML to be the plain (faster, less readable) form
+            RDFFormat fmt = 
+                ( lang == Lang.RDFXML ) ? RDFFormat.RDFXML_PLAIN : RDFWriterRegistry.defaultSerialization(lang) ;  
+            RDFDataMgr.write(out, model, fmt) ;
             success(action) ;
         } finally { action.endRead() ; }
     }