[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 <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Reviewed-by: Dmitry Lychagin <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/asterixdb/repo Commit: http://git-wip-us.apache.org/repos/asf/asterixdb/commit/36e20b78 Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/36e20b78 Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/36e20b78 Branch: refs/heads/master Commit: 36e20b78bb1584538c9844e2e7398eacf8dd3fb8 Parents: 5e24a1b Author: luochen01 <[email protected]> Authored: Fri Sep 14 09:31:54 2018 -0700 Committer: Dmitry Lychagin <[email protected]> Committed: Fri Sep 14 11:34:31 2018 -0700 ---------------------------------------------------------------------- .../core/algebra/properties/UnorderedPartitionedProperty.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/asterixdb/blob/36e20b78/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/UnorderedPartitionedProperty.java ---------------------------------------------------------------------- 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 @@ public final class UnorderedPartitionedProperty extends AbstractGroupingProperty @Override public IPartitioningProperty clonePartitioningProperty() { - return new UnorderedPartitionedProperty(new HashSet<>(columnSet), domain); + return new UnorderedPartitionedProperty(new ListSet<>(columnSet), domain); } }
