Merge pull request apache#3422 from wang3820/fix-flaky-databaselookup

Fixed flaky test due to use of HashSet in DatabaseLookupMetaTest#testProvidesModelerMeta
hansva · Nov 21, 2023 · 1a276ed · 1a276ed
2 parents 96cfd6a + 81eb002
commit 1a276ed
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...p/src/main/java/org/apache/hop/pipeline/transforms/databaselookup/DatabaseLookupMeta.java b/...p/src/main/java/org/apache/hop/pipeline/transforms/databaselookup/DatabaseLookupMeta.java
@@ -44,7 +44,7 @@
 import org.apache.hop.pipeline.transform.TransformMeta;
 
 import java.util.ArrayList;
-import java.util.HashSet;
+import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Set;
 
@@ -444,7 +444,7 @@ public String getSchemaName() {
 
   @Override
   public List<String> getDatabaseFields() {
-    Set<String> fields = new HashSet<>();
+    Set<String> fields = new LinkedHashSet<>();
     for (KeyField keyField : lookup.getKeyFields()) {
       if (StringUtils.isNotEmpty(keyField.getTableField())) {
         fields.add(keyField.getTableField());
@@ -460,7 +460,7 @@ public List<String> getDatabaseFields() {
 
   @Override
   public List<String> getStreamFields() {
-    Set<String> fields = new HashSet<>();
+    Set<String> fields = new LinkedHashSet<>();
     for (KeyField keyField : lookup.getKeyFields()) {
       if (StringUtils.isNotEmpty(keyField.getStreamField1())) {
         fields.add(keyField.getStreamField1());