[calcite] branch master updated: [CALCITE-3389] Test may fail if HashSet iterates in different order (contextshuffling)

[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 abf36e4  [CALCITE-3389] Test may fail if HashSet iterates in different 
order (contextshuffling)
abf36e4 is described below

commit abf36e4da4884f2c22435384aa55d68f968a784e
Author: contextshuffling <contextshuffl...@gmail.com>
AuthorDate: Sat Oct 5 15:45:35 2019 -0500

    [CALCITE-3389] Test may fail if HashSet iterates in different order 
(contextshuffling)
    
    Close #1487
---
 linq4j/src/test/java/org/apache/calcite/linq4j/test/ExpressionTest.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/linq4j/src/test/java/org/apache/calcite/linq4j/test/ExpressionTest.java 
b/linq4j/src/test/java/org/apache/calcite/linq4j/test/ExpressionTest.java
index 005e8ae..c65e076 100644
--- a/linq4j/src/test/java/org/apache/calcite/linq4j/test/ExpressionTest.java
+++ b/linq4j/src/test/java/org/apache/calcite/linq4j/test/ExpressionTest.java
@@ -1536,7 +1536,7 @@ public class ExpressionTest {
   }
 
   @Test public void testTenElementsSetLiteral() throws Exception {
-    Set set = new HashSet(); // for consistent output
+    Set set = new LinkedHashSet(); // for consistent output
     for (int i = 0; i < 10; i++) {
       set.add(i);
     }