[GitHub] [pinot] ankitsultana commented on a diff in pull request #10758: Add a new controller endpoint for segment deletion with a time window

["ankitsultana (via GitHub)" <gi...@apache.org>]

ankitsultana commented on code in PR #10758:
URL: https://github.com/apache/pinot/pull/10758#discussion_r1228820464


##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java:
##########
@@ -884,6 +865,47 @@ public SuccessResponse deleteSegments(
     }
   }
 
+  @DELETE
+  @Consumes(MediaType.APPLICATION_JSON)
+  @Produces(MediaType.APPLICATION_JSON)
+  @Path("/segments/{tableName}/select")

Review Comment:
   We want to be able to delete data older than a given age using an API. At present I think we only have APIs that either delete all segments or delete one particular segment.
   
   This functionality should be useful to everyone since this is a common requirement when you have a lot of customers using Pinot.



-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org