You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by "rameeshm (via GitHub)" <gi...@apache.org> on 2026/05/08 07:43:04 UTC
[PR] RANGER-5577:API to support bulk delete of resources in DataShare [ranger]
rameeshm opened a new pull request, #936:
URL: https://github.com/apache/ranger/pull/936
## What changes were proposed in this pull request?
To have the API to support bulk delete by getting the resources Id as query param.
## How was this patch tested?
Test in local via with curl call.
` curl -ikv --negotiate -u : -H "Accept: application/json" -H "Content-Type: application/json" -X GET 'https://<ranger-admin-host>:6182/service/gds/resource?datasetId=2'`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ranger.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RANGER-5577:API to support bulk delete of resources in DataShare [ranger]
Posted by "mneethiraj (via GitHub)" <gi...@apache.org>.
mneethiraj commented on code in PR #936:
URL: https://github.com/apache/ranger/pull/936#discussion_r3209865446
##########
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) {
Review Comment:
@rameeshm - given a separate query parameter will be needed for each resource, consider naming the query parameter as `resourceId` or `id`. I suggest `id` as its shorter, hence can handle larger number of resources.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ranger.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RANGER-5577:API to support bulk delete of resources in DataShare [ranger]
Posted by "rameeshm (via GitHub)" <gi...@apache.org>.
rameeshm merged PR #936:
URL: https://github.com/apache/ranger/pull/936
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@ranger.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org