Steven Jacobs has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/2730
Change subject: [COMP] check head size before reducing grouping columns
......................................................................
[COMP] check head size before reducing grouping columns
Change-Id: Id2a6a16a5342e30c246b62a3b5ddbf7249a0d0cc
---
M
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/AbstractGroupingProperty.java
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/30/2730/1
diff --git
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/AbstractGroupingProperty.java
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/AbstractGroupingProperty.java
index 0d8d911..60b5bc1 100644
---
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/AbstractGroupingProperty.java
+++
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/AbstractGroupingProperty.java
@@ -82,7 +82,7 @@
for (LogicalVariable v : columnSet) {
boolean isImpliedByAnFD = false;
for (FunctionalDependency fdep : fds) {
- if (columnSet.containsAll(fdep.getHead()) &&
fdep.getTail().contains(v)) {
+ if (fdep.getHead().size() > 0 &&
columnSet.containsAll(fdep.getHead()) && fdep.getTail().contains(v)) {
isImpliedByAnFD = true;
norm.addAll(fdep.getHead());
break;
--
To view, visit https://asterix-gerrit.ics.uci.edu/2730
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id2a6a16a5342e30c246b62a3b5ddbf7249a0d0cc
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Steven Jacobs <[email protected]>
