You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@quickstep.apache.org by sh...@apache.org on 2016/07/12 19:49:18 UTC

incubator-quickstep git commit: Minor fix

Repository: incubator-quickstep
Updated Branches:
  refs/heads/SQL-window-aggregation f4eb4a7cc -> e16fdf752


Minor fix


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

Branch: refs/heads/SQL-window-aggregation
Commit: e16fdf75240a705a6d18a841cf94bcd78619f077
Parents: f4eb4a7
Author: shixuan-fan <sh...@apache.org>
Authored: Tue Jul 12 19:48:59 2016 +0000
Committer: shixuan-fan <sh...@apache.org>
Committed: Tue Jul 12 19:48:59 2016 +0000

----------------------------------------------------------------------
 storage/WindowAggregationOperationState.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-quickstep/blob/e16fdf75/storage/WindowAggregationOperationState.cpp
----------------------------------------------------------------------
diff --git a/storage/WindowAggregationOperationState.cpp b/storage/WindowAggregationOperationState.cpp
index 06e846f..485ca47 100644
--- a/storage/WindowAggregationOperationState.cpp
+++ b/storage/WindowAggregationOperationState.cpp
@@ -72,10 +72,9 @@ WindowAggregationOperationState::WindowAggregationOperationState(
   DCHECK(window_aggregate_function->canApplyToTypes(argument_types));
 
   // IDs and types of partition keys.
-  std::vector<attribute_id> partition_by_ids;
   std::vector<const Type*> partition_by_types;
   for (const std::unique_ptr<const Scalar> &partition_by_attribute : partition_by_attributes) {
-    partition_by_ids.push_back(
+    partition_by_ids_.push_back(
         partition_by_attribute->getAttributeIdForValueAccessor());
     partition_by_types.push_back(&partition_by_attribute->getType());
   }