[calcite] branch master updated: [CALCITE-3947] AbstractRelOptPlanner.classes should be LinkedHashSet so that rule match order is deterministic across runs

[hyuan]

This is an automated email from the ASF dual-hosted git repository.

hyuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/master by this push:
     new 53bbd2d  [CALCITE-3947] AbstractRelOptPlanner.classes should be 
LinkedHashSet so that rule match order is deterministic across runs
53bbd2d is described below

commit 53bbd2daff8104f3a1113eaaa01bf58ca3361beb
Author: botong.huang <botong.hu...@alibaba-inc.com>
AuthorDate: Sat Apr 18 21:03:01 2020 -0700

    [CALCITE-3947] AbstractRelOptPlanner.classes should be LinkedHashSet so 
that rule match order is deterministic across runs
---
 core/src/main/java/org/apache/calcite/plan/AbstractRelOptPlanner.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/core/src/main/java/org/apache/calcite/plan/AbstractRelOptPlanner.java 
b/core/src/main/java/org/apache/calcite/plan/AbstractRelOptPlanner.java
index 02dcce2..3d1487f 100644
--- a/core/src/main/java/org/apache/calcite/plan/AbstractRelOptPlanner.java
+++ b/core/src/main/java/org/apache/calcite/plan/AbstractRelOptPlanner.java
@@ -32,6 +32,7 @@ import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashSet;
 import java.util.LinkedHashMap;
+import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
@@ -60,7 +61,7 @@ public abstract class AbstractRelOptPlanner implements 
RelOptPlanner {
 
   protected final AtomicBoolean cancelFlag;
 
-  private final Set<Class<? extends RelNode>> classes = new HashSet<>();
+  private final Set<Class<? extends RelNode>> classes = new LinkedHashSet<>();
 
   private final Set<Convention> conventions = new HashSet<>();