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 2015/06/26 20:07:04 UTC
[2/3] jena git commit: Copy aggregators in a shallow copy.
Copy aggregators in a shallow copy.
Project: http://git-wip-us.apache.org/repos/asf/jena/repo
Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/4fb9bca9
Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/4fb9bca9
Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/4fb9bca9
Branch: refs/heads/master
Commit: 4fb9bca9d813a98ea0604a7ff6fbadbd9c327be2
Parents: 820714b
Author: Andy Seaborne <an...@apache.org>
Authored: Fri Jun 26 18:41:47 2015 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Fri Jun 26 18:41:47 2015 +0100
----------------------------------------------------------------------
.../syntax/syntaxtransform/QueryTransformOps.java | 12 +++---------
1 file changed, 3 insertions(+), 9 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/jena/blob/4fb9bca9/jena-arq/src/main/java/org/apache/jena/sparql/syntax/syntaxtransform/QueryTransformOps.java
----------------------------------------------------------------------
diff --git a/jena-arq/src/main/java/org/apache/jena/sparql/syntax/syntaxtransform/QueryTransformOps.java b/jena-arq/src/main/java/org/apache/jena/sparql/syntax/syntaxtransform/QueryTransformOps.java
index b118cf9..c05823a 100644
--- a/jena-arq/src/main/java/org/apache/jena/sparql/syntax/syntaxtransform/QueryTransformOps.java
+++ b/jena-arq/src/main/java/org/apache/jena/sparql/syntax/syntaxtransform/QueryTransformOps.java
@@ -122,15 +122,9 @@ public class QueryTransformOps {
for (String x : desc.getDefaultGraphURIs())
newQuery.addNamedGraphURI(x) ;
}
- // Not needed in a shallow copy?
- // for ( ExprAggregator agg : query.getAggregators() )
- // {
- // // Hooks for
- // //aggregatorsAllocated.put(key, v) ;
- // //aggregatorsMap.put(v, aggExpr) ;
- // //aggregators.add(aggExpr) ;
- // }
-
+
+ // Aggregators.
+ newQuery.getAggregators().addAll(query.getAggregators()) ;
}
@Override