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/03/21 21:12:19 UTC
svn commit: r1459505 -
/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java
Author: andy
Date: Thu Mar 21 20:12:18 2013
New Revision: 1459505
URL: http://svn.apache.org/r1459505
Log:
Fix comments
Modified:
jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java
Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java?rev=1459505&r1=1459504&r2=1459505&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/query/Query.java Thu Mar 21 20:12:18 2013
@@ -531,7 +531,7 @@ public class Query extends Prologue impl
// Record allocated aggregations.
// Later: The same aggregation expression used in a query
// will always lead to the same aggregator.
- // For now, allocate a fresh one each time (cause the calcutation
+ // For now, allocate a fresh one each time (cause the calculation
// to be done multiple times but (1) it's unusual to have repeated
// aggregators normally and (2) the actual calculation is cheap.
@@ -539,7 +539,6 @@ public class Query extends Prologue impl
// Commonality?
private List<ExprAggregator> aggregators = new ArrayList<ExprAggregator>() ;
- // Using a LinkedhashMap does seem to give strong enugh hashCode equality.
private Map<Var, ExprAggregator> aggregatorsMap = new HashMap<Var, ExprAggregator>() ;
// Note any E_Aggregator created for reuse.
@@ -551,7 +550,7 @@ public class Query extends Prologue impl
public Expr allocAggregate(Aggregator agg)
{
// We need to track the aggregators in case one aggregator is used twice, e.g. in HAVING and in SELECT expression
- // (is is that much harm to do twice? Yes, if distinct.)
+ // (is that much harm to do twice? Yes, if distinct.)
String key = agg.key() ;
Var v = aggregatorsAllocated.get(key);