[New] Logs computed model if isReplace=false

kbss-cvut · Nov 21, 2021 · 559c528 · 559c528
1 parent 908b708
commit 559c528
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 39 deletions.
diff --git a/s-pipes-core/src/main/java/cz/cvut/spipes/modules/AbstractModule.java b/s-pipes-core/src/main/java/cz/cvut/spipes/modules/AbstractModule.java
@@ -7,6 +7,7 @@
 import cz.cvut.spipes.engine.ExecutionContextFactory;
 import cz.cvut.spipes.engine.VariablesBinding;
 import cz.cvut.spipes.exception.ValidationConstraintFailed;
+import cz.cvut.spipes.util.JenaUtils;
 import org.apache.jena.atlas.lib.NotImplemented;
 import org.apache.jena.query.Query;
 import org.apache.jena.query.QueryExecution;
@@ -350,6 +351,17 @@ protected RDFNode getEffectiveValue(@NotNull Property valueProperty) {
         }
     }
 
+    protected ExecutionContext createOutputContext(boolean isReplace, Model inputModel, Model computedModel) {
+        if (isReplace) {
+            return ExecutionContextFactory.createContext(computedModel);
+        } else {
+            if (AuditConfig.isEnabled() || isInDebugMode) {
+                LOG.debug("Saving module computed output to file {}.", saveModelToTemporaryFile(outputContext.getDefaultModel()));
+            }
+            return ExecutionContextFactory.createContext(JenaUtils.createUnion(inputModel, computedModel));
+        }
+    }
+
 //    @Override
 //    public String toString() {
 //        String resourceId = (resource  != null) ? ( " (" + resource.getId() + ")") : "";

diff --git a/s-pipes-core/src/main/java/cz/cvut/spipes/modules/ApplyConstructModule.java b/s-pipes-core/src/main/java/cz/cvut/spipes/modules/ApplyConstructModule.java
@@ -3,7 +3,6 @@
 import cz.cvut.spipes.constants.KBSS_MODULE;
 import cz.cvut.spipes.constants.SML;
 import cz.cvut.spipes.engine.ExecutionContext;
-import cz.cvut.spipes.engine.ExecutionContextFactory;
 import cz.cvut.spipes.util.JenaUtils;
 import cz.cvut.spipes.util.QueryUtils;
 import org.apache.jena.query.Query;
@@ -144,20 +143,9 @@ public ExecutionContext executeSelf() {
             nNew = inferredInSingleIterationModel.size();
         }
 
-        if (isReplace) {
-            return ExecutionContextFactory.createContext(inferredModel);
-        } else {
-            return ExecutionContextFactory.createContext(JenaUtils.createUnion(defaultModel, inferredModel));
-        }
+        return this.createOutputContext(isReplace, defaultModel, inferredModel);
     }
 
-    // TODO move this to external utils
-
-
-
-
-
-
     @Override
     public String getTypeURI() {
         return SML.ApplyConstruct.getURI();

diff --git a/s-pipes-core/src/main/java/cz/cvut/spipes/modules/SemFormGeneratorModule.java b/s-pipes-core/src/main/java/cz/cvut/spipes/modules/SemFormGeneratorModule.java