Fixing gradle errors

Signed-off-by: Bharathwaj G <[email protected]>
opensearch-project · Jun 27, 2022 · d4ecca2 · d4ecca2
1 parent 17a824e
commit d4ecca2
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 7 deletions.
diff --git a/rest-api-spec/src/main/resources/rest-api-spec/api/create_pit.json b/rest-api-spec/src/main/resources/rest-api-spec/api/create_pit.json
@@ -4,7 +4,7 @@
       "url":"https://opensearch.org/docs/latest/opensearch/rest-api/point_in_time/",
       "description":"Creates point in time context."
     },
-    "stability":"stable",
+    "stability":"experimental",
     "url":{
       "paths":[
         {

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/api/delete_all_pits.json b/rest-api-spec/src/main/resources/rest-api-spec/api/delete_all_pits.json
@@ -4,7 +4,7 @@
       "url":"https://opensearch.org/docs/latest/opensearch/rest-api/point_in_time/",
       "description":"Deletes all active point in time searches."
     },
-    "stability":"stable",
+    "stability":"experimental",
     "url":{
       "paths":[
         {

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/api/delete_pit.json b/rest-api-spec/src/main/resources/rest-api-spec/api/delete_pit.json
@@ -4,7 +4,7 @@
       "url":"https://opensearch.org/docs/latest/opensearch/rest-api/point_in_time/",
       "description":"Deletes one or more point in time searches based on the IDs passed."
     },
-    "stability":"stable",
+    "stability":"experimental",
     "url":{
       "paths":[
         {

diff --git a/rest-api-spec/src/main/resources/rest-api-spec/test/pit/10_basic.yml b/rest-api-spec/src/main/resources/rest-api-spec/test/pit/10_basic.yml
@@ -1,4 +1,7 @@
 "Create PIT, Search with PIT ID and Delete":
+  - skip:
+      version: " - 2.9.99"
+      reason: "mode to be introduced later than 3.0"
   - do:
       indices.create:
         index: test_pit
@@ -88,6 +91,9 @@
 
 ---
 "Delete all":
+  - skip:
+      version: " - 2.9.99"
+      reason: "mode to be introduced later than 3.0"
   - do:
       indices.create:
         index: test_pit
@@ -124,6 +130,5 @@
   - match: {pits.0.succeeded: true }
 
   - do:
+      catch: missing
       delete_all_pits: { }
-
-  - match: { pits: []}
diff --git a/server/src/main/java/org/opensearch/action/search/DeletePitResponse.java b/server/src/main/java/org/opensearch/action/search/DeletePitResponse.java
@@ -58,7 +58,7 @@ public List<DeletePitInfo> getDeletePitResults() {
 
     @Override
     public RestStatus status() {
-        if(deletePitResults.isEmpty()) return NOT_FOUND;
+        if (deletePitResults.isEmpty()) return NOT_FOUND;
         return OK;
     }
 

diff --git a/server/src/test/java/org/opensearch/search/CreatePitSingleNodeTests.java b/server/src/test/java/org/opensearch/search/CreatePitSingleNodeTests.java
@@ -228,7 +228,7 @@ public void testSearchWithFirstPhaseKeepAliveExpiry() throws ExecutionException,
         // since first phase temporary keep alive is set at 1 second in this test file
         // and create pit request keep alive is less than that, keep alive is set to 1 second, (max of 2 keep alives)
         // so reader context will clear up after 1 second
-        Thread.sleep(1000);
+        Thread.sleep(1200);
         client().prepareIndex("index").setId("2").setSource("field", "value").setRefreshPolicy(IMMEDIATE).get();
 
         SearchPhaseExecutionException ex = expectThrows(SearchPhaseExecutionException.class, () -> {