Dmitry Lychagin has submitted this change and it was merged. Change subject: [ASTERIXDB-2174] Use ListSet when clone partition property ......................................................................
[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]> --- M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/properties/UnorderedPartitionedProperty.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Anon. E. Moose #1000171: Jenkins: Verified; No violations found; Verified Dmitry Lychagin: Looks good to me, approved 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); } } -- To view, visit https://asterix-gerrit.ics.uci.edu/2968 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2b8c1715d75b387a73504f4709524d9ab9ce123 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Luo Chen <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]>
