addrssed all comment

Signed-off-by: Junwei Dai <[email protected]>
opensearch-project · Nov 1, 2024 · 0c73128 · 0c73128
1 parent 96bcd36
commit 0c73128
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/test-api-consistency.yml b/.github/workflows/test-api-consistency.yml
@@ -3,6 +3,7 @@ name: Daily API Consistency Test
 on:
   schedule:
     - cron: '0 8 * * *'  # Runs daily at 8 AM UTC
+  workflow_dispatch:
 
 jobs:
   API-consistency-test:

diff --git a/src/main/java/org/opensearch/flowframework/util/ApiSpecFetcher.java b/src/main/java/org/opensearch/flowframework/util/ApiSpecFetcher.java
@@ -14,7 +14,6 @@
 import org.opensearch.flowframework.exception.ApiSpecParseException;
 import org.opensearch.rest.RestRequest;
 
-import java.util.HashSet;
 import java.util.List;
 
 import io.swagger.v3.oas.models.OpenAPI;
@@ -81,7 +80,7 @@ public static boolean compareRequiredFields(List<String> requiredEnumParams, Str
 
             List<String> requiredApiParams = schema.getRequired();
             if (requiredApiParams != null && !requiredApiParams.isEmpty()) {
-                return new HashSet<>(requiredEnumParams).containsAll(new HashSet<>(requiredApiParams));
+                return requiredApiParams.stream().allMatch(requiredEnumParams::contains);
             }
         }
         return false;