You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@marmotta.apache.org by ss...@apache.org on 2014/11/09 18:15:05 UTC

marmotta git commit: SPARQL: - add comment for documentation

Repository: marmotta
Updated Branches:
  refs/heads/develop 90620e29a -> eda42c246


SPARQL:
- add comment for documentation


Project: http://git-wip-us.apache.org/repos/asf/marmotta/repo
Commit: http://git-wip-us.apache.org/repos/asf/marmotta/commit/eda42c24
Tree: http://git-wip-us.apache.org/repos/asf/marmotta/tree/eda42c24
Diff: http://git-wip-us.apache.org/repos/asf/marmotta/diff/eda42c24

Branch: refs/heads/develop
Commit: eda42c246a83692031b42ae7d22e9497a6b27dc5
Parents: 90620e2
Author: Sebastian Schaffert <ss...@apache.org>
Authored: Sun Nov 9 18:15:44 2014 +0100
Committer: Sebastian Schaffert <ss...@apache.org>
Committed: Sun Nov 9 18:15:44 2014 +0100

----------------------------------------------------------------------
 .../java/org/apache/marmotta/kiwi/sparql/builder/SQLBuilder.java  | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/marmotta/blob/eda42c24/libraries/kiwi/kiwi-sparql/src/main/java/org/apache/marmotta/kiwi/sparql/builder/SQLBuilder.java
----------------------------------------------------------------------
diff --git a/libraries/kiwi/kiwi-sparql/src/main/java/org/apache/marmotta/kiwi/sparql/builder/SQLBuilder.java b/libraries/kiwi/kiwi-sparql/src/main/java/org/apache/marmotta/kiwi/sparql/builder/SQLBuilder.java
index ac24190..0f9cbbb 100644
--- a/libraries/kiwi/kiwi-sparql/src/main/java/org/apache/marmotta/kiwi/sparql/builder/SQLBuilder.java
+++ b/libraries/kiwi/kiwi-sparql/src/main/java/org/apache/marmotta/kiwi/sparql/builder/SQLBuilder.java
@@ -367,6 +367,9 @@ public class SQLBuilder {
 
                         // if the variable has been used before, add a join condition to the first occurrence
                         if(sv.getExpressions().size() > 0) {
+                            // case distinction: is this variable projected as node or as another value in an extension?
+                            // if it is a value, we need to refer to the corresponding typed column of the node, otherwise
+                            // to the node ID (field ID is sufficient)
                             switch (sv.getProjectionType()) {
                                 case INT:
                                     p.getConditions().add(sv.getExpressions().get(0) + " = " + sv.getAlias() + ".ivalue");