RANGER-5577:API to support bulk delete of resources in DataShare

Author: Ramesh Mani

security-admin/src/main/java/org/apache/ranger/rest/GdsREST.java

@@ -1292,12 +1292,16 @@ public void removeSharedResource(@PathParam("id") Long resourceId) {
     @DELETE
     @Path("/resources")
     @PreAuthorize("@rangerPreAuthSecurityHandler.isAPIAccessible(\"" + RangerAPIList.REMOVE_SHARED_RESOURCES + "\")")
-    public void removeSharedResources(List<Long> resourceIds) {
-        LOG.debug("==> GdsREST.removeSharedResources({})", resourceIds);
+    public void removeSharedResources(@QueryParam("resourceIds") List<Long> resourceIds) {
+        LOG.debug("==> GdsREST.removeSharedResources(resourceIds={})", resourceIds);
 
         RangerPerfTracer perf = null;
 
         try {
+            if (resourceIds == null) {
+                throw new Exception("resourceIds must not be null");
+            }
+
             if (resourceIds.size() > SHARED_RESOURCES_MAX_BATCH_SIZE) {
                 throw new Exception("removeSharedResources batch size exceeded the configured limit: Maximum allowed is " + SHARED_RESOURCES_MAX_BATCH_SIZE);
             }
@@ -1317,7 +1321,7 @@ public void removeSharedResources(List<Long> resourceIds) {
             RangerPerfTracer.log(perf);
         }
 
-        LOG.debug("<== GdsREST.removeSharedResources({})", resourceIds);
+        LOG.debug("<== GdsREST.removeSharedResources(resourceIds={})", resourceIds);
     }
 
     @GET

security-admin/src/test/java/org/apache/ranger/rest/TestGdsREST.java

@@ -962,6 +962,13 @@ public void testRemoveSharedResourcesBatchSizeExceeded() {
         assertThrows(WebApplicationException.class, () -> gdsREST.removeSharedResources(resourceIds));
     }
 
+    @Test
+    public void testRemoveSharedResourcesNullResourceIds() {
+        Mockito.when(restErrorUtil.createRESTException(Mockito.anyString())).thenReturn(new WebApplicationException());
+
+        assertThrows(WebApplicationException.class, () -> gdsREST.removeSharedResources(null));
+    }
+
     @Test
     public void testGetSharedResource() {
         Long resourceId = 1L;