-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding instant deletion option for segment deletion #8122
adding instant deletion option for segment deletion #8122
Conversation
8c566e5
to
61076f5
Compare
@@ -571,12 +571,15 @@ public SuccessResponse reloadAllSegmentsDeprecated2( | |||
@ApiOperation(value = "Delete a segment", notes = "Delete a segment") | |||
public SuccessResponse deleteSegment( | |||
@ApiParam(value = "Name of the table", required = true) @PathParam("tableName") String tableName, | |||
@ApiParam(value = "Name of the segment", required = true) @PathParam("segmentName") @Encoded String segmentName) { | |||
@ApiParam(value = "Name of the segment", required = true) @PathParam("segmentName") @Encoded String segmentName, | |||
@ApiParam(value = "Whether to delete the segment instantly or move to deleted_segment prefix and let " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ApiParam(value = "Whether to delete the segment instantly or move to deleted_segment prefix and let " | |
@ApiParam(value = "Whether to delete the segment without backup" |
@ApiParam(value = "Name of the segment", required = true) @PathParam("segmentName") @Encoded String segmentName) { | ||
@ApiParam(value = "Name of the segment", required = true) @PathParam("segmentName") @Encoded String segmentName, | ||
@ApiParam(value = "Whether to delete the segment instantly or move to deleted_segment prefix and let " | ||
+ "RetentionManager handle the actual file deletion") @QueryParam("instantDelete") @DefaultValue("false") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+ "RetentionManager handle the actual file deletion") @QueryParam("instantDelete") @DefaultValue("false") | |
) @QueryParam("deleteWithoutBackup") @DefaultValue("false") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with #8176 we will be able to change the API to accept retention period, which was what you originally suggested
+ "RetentionManager handle the actual file deletion") @QueryParam("instantDelete") @DefaultValue("false") | |
+ @ApiParam(value = "segment delete retention period, (for example 12h, 3d); setting it to 0d will instant delete the segments") @QueryParam("deleteRetention") @DefaultValue("7d") deleteRetentionPeriod |
Codecov Report
@@ Coverage Diff @@
## master #8122 +/- ##
============================================
+ Coverage 71.01% 71.07% +0.06%
Complexity 4320 4320
============================================
Files 1626 1626
Lines 85067 85072 +5
Branches 12799 12800 +1
============================================
+ Hits 60408 60468 +60
+ Misses 20505 20449 -56
- Partials 4154 4155 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
61076f5
to
0949a1b
Compare
Add deletion with retention period overwrite for segment deletion
per discussion in #8072, #8078 and #8176
We will add deletion with retention period overwrite for segment deletion.