You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@quickstep.apache.org by hb...@apache.org on 2016/08/19 15:35:46 UTC
[4/5] incubator-quickstep git commit: Minor change in DCHECK.
Minor change in DCHECK.
Project: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/commit/5b4b0ca4
Tree: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/tree/5b4b0ca4
Diff: http://git-wip-us.apache.org/repos/asf/incubator-quickstep/diff/5b4b0ca4
Branch: refs/heads/partitioned-aggregation
Commit: 5b4b0ca412fe59c55a7da996a9cf6841536fbf00
Parents: 104fa78
Author: Harshad Deshmukh <hb...@apache.org>
Authored: Thu Aug 18 22:14:09 2016 -0500
Committer: Harshad Deshmukh <hb...@apache.org>
Committed: Thu Aug 18 22:14:09 2016 -0500
----------------------------------------------------------------------
storage/StorageBlock.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-quickstep/blob/5b4b0ca4/storage/StorageBlock.cpp
----------------------------------------------------------------------
diff --git a/storage/StorageBlock.cpp b/storage/StorageBlock.cpp
index cdab0ff..6c4c36b 100644
--- a/storage/StorageBlock.cpp
+++ b/storage/StorageBlock.cpp
@@ -1458,7 +1458,7 @@ void StorageBlock::aggregateGroupByPartitioned(
std::unique_ptr<TupleIdSequence> *reuse_matches,
std::vector<std::unique_ptr<ColumnVector>> *reuse_group_by_vectors,
PartitionedHashTablePool *hashtable_pool) const {
- DCHECK_EQ(group_by.size(), 0u)
+ DCHECK(!group_by.empty())
<< "Called aggregateGroupByPartitioned() with zero GROUP BY expressions";
SubBlocksReference sub_blocks_ref(*tuple_store_,