From 53e94c8c35aba50a4e571897bfe76682db15273e Mon Sep 17 00:00:00 2001 From: "daniel.cui" Date: Tue, 20 Dec 2011 14:52:16 +0800 Subject: [PATCH] Checkin:Using a formula as a group binding does not work correctly.[45785] --- .../data/engine/executor/QueryExecutionStrategyUtil.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/data/org.eclipse.birt.data/src/org/eclipse/birt/data/engine/executor/QueryExecutionStrategyUtil.java b/data/org.eclipse.birt.data/src/org/eclipse/birt/data/engine/executor/QueryExecutionStrategyUtil.java index 3c51e757c3d..898106b2e00 100644 --- a/data/org.eclipse.birt.data/src/org/eclipse/birt/data/engine/executor/QueryExecutionStrategyUtil.java +++ b/data/org.eclipse.birt.data/src/org/eclipse/birt/data/engine/executor/QueryExecutionStrategyUtil.java @@ -111,6 +111,11 @@ public static Strategy getQueryExecutionStrategy( DataEngineSession session, IQu } boolean hasAggregation = false; + boolean hasGroup = false; + if( query.getGroups()!=null && query.getGroups().size()>0) + { + hasGroup = true; + } if ( query.getBindings( ) != null ) { @@ -209,6 +214,6 @@ public static Strategy getQueryExecutionStrategy( DataEngineSession session, IQu } } - return hasAggregation?Strategy.SimpleLookingFoward:Strategy.SimpleNoLookingFoward; + return (hasAggregation||hasGroup)?Strategy.SimpleLookingFoward:Strategy.SimpleNoLookingFoward; } }