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

Author: Ramesh Mani

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

@@ -1292,36 +1292,36 @@ public void removeSharedResource(@PathParam("id") Long resourceId) {
     @DELETE
     @Path("/resources")
     @PreAuthorize("@rangerPreAuthSecurityHandler.isAPIAccessible(\"" + RangerAPIList.REMOVE_SHARED_RESOURCES + "\")")
-    public void removeSharedResources(@QueryParam("resourceIds") List<Long> resourceIds) {
-        LOG.debug("==> GdsREST.removeSharedResources(resourceIds={})", resourceIds);
+    public void removeSharedResources(@QueryParam("id") List<Long> id) {
+        LOG.debug("==> GdsREST.removeSharedResources(resourceIds={})", id);
 
         RangerPerfTracer perf = null;
 
         try {
-            if (resourceIds == null) {
+            if (id == null) {
                 throw new Exception("resourceIds must not be null");
             }
 
-            if (resourceIds.size() > SHARED_RESOURCES_MAX_BATCH_SIZE) {
+            if (id.size() > SHARED_RESOURCES_MAX_BATCH_SIZE) {
                 throw new Exception("removeSharedResources batch size exceeded the configured limit: Maximum allowed is " + SHARED_RESOURCES_MAX_BATCH_SIZE);
             }
 
             if (RangerPerfTracer.isPerfTraceEnabled(PERF_LOG)) {
-                perf = RangerPerfTracer.getPerfTracer(PERF_LOG, "GdsREST.removeSharedResources(" + resourceIds + ")");
+                perf = RangerPerfTracer.getPerfTracer(PERF_LOG, "GdsREST.removeSharedResources(" + id + ")");
             }
 
-            gdsStore.removeSharedResources(resourceIds);
+            gdsStore.removeSharedResources(id);
         } catch (WebApplicationException excp) {
             throw excp;
         } catch (Throwable excp) {
-            LOG.error("removeSharedResources({}) failed", resourceIds, excp);
+            LOG.error("removeSharedResources({}) failed", id, excp);
 
             throw restErrorUtil.createRESTException(excp.getMessage());
         } finally {
             RangerPerfTracer.log(perf);
         }
 
-        LOG.debug("<== GdsREST.removeSharedResources(resourceIds={})", resourceIds);
+        LOG.debug("<== GdsREST.removeSharedResources(resourceIds={})", id);
     }
 
     @GET