[ASTERIXDB-2174] Use ListSet when clone partition property
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
- Use ListSet instead of HashSet when cloning partitioning properties
to ensure it has the consistent behavior as before.
Change-Id: Ib2b8c1715d75b387a73504f4709524d9ab9ce123
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2968
Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Dmitry Lychagin <dmitry.lychagin@couchbase.com>
diff --git a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/UnorderedPartitionedProperty.java b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/UnorderedPartitionedProperty.java
index e07cf15..5966407 100644
--- a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/UnorderedPartitionedProperty.java
+++ b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/UnorderedPartitionedProperty.java
@@ -19,11 +19,11 @@
package org.apache.hyracks.algebricks.core.algebra.properties;
import java.util.Collection;
-import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;
+import org.apache.hyracks.algebricks.common.utils.ListSet;
import org.apache.hyracks.algebricks.core.algebra.base.EquivalenceClass;
import org.apache.hyracks.algebricks.core.algebra.base.LogicalVariable;
@@ -80,7 +80,7 @@
@Override
public IPartitioningProperty clonePartitioningProperty() {
- return new UnorderedPartitionedProperty(new HashSet<>(columnSet), domain);
+ return new UnorderedPartitionedProperty(new ListSet<>(columnSet), domain);
}
}