You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hawq.apache.org by en...@apache.org on 2015/12/18 20:22:03 UTC

incubator-hawq git commit: HAWQ-264. Fix coverity issue 16858

Repository: incubator-hawq
Updated Branches:
  refs/heads/master b6cb39c37 -> 8239cb9a1


HAWQ-264. Fix coverity issue 16858


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

Branch: refs/heads/master
Commit: 8239cb9a108e97a8ef48b717526561bf83d158c1
Parents: b6cb39c
Author: Entong Shen <es...@pivotal.io>
Authored: Fri Dec 18 11:15:57 2015 -0800
Committer: Entong Shen <sh...@gmail.com>
Committed: Fri Dec 18 11:20:37 2015 -0800

----------------------------------------------------------------------
 src/backend/gpopt/translate/CTranslatorQueryToDXL.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-hawq/blob/8239cb9a/src/backend/gpopt/translate/CTranslatorQueryToDXL.cpp
----------------------------------------------------------------------
diff --git a/src/backend/gpopt/translate/CTranslatorQueryToDXL.cpp b/src/backend/gpopt/translate/CTranslatorQueryToDXL.cpp
index 946e0ae..be9c077 100644
--- a/src/backend/gpopt/translate/CTranslatorQueryToDXL.cpp
+++ b/src/backend/gpopt/translate/CTranslatorQueryToDXL.cpp
@@ -2230,9 +2230,10 @@ CTranslatorQueryToDXL::PdxlnUnionAllForGroupingSets
 	)
 {
 	GPOS_ASSERT(NULL != pdrgpbs);
-	GPOS_ASSERT(1 < pdrgpbs->UlLength());
 
 	const ULONG ulGroupingSets = pdrgpbs->UlLength();
+	GPOS_ASSERT(1 < ulGroupingSets);
+
 	CDXLNode *pdxlnUnionAll = NULL;
 	DrgPul *pdrgpulColIdsInner = NULL;