abhishekrb19 commented on code in PR #15710:
URL: https://github.com/apache/druid/pull/15710#discussion_r1456897016
##########
server/src/main/java/org/apache/druid/rpc/indexing/OverlordClient.java:
##########
@@ -72,21 +73,28 @@ public interface OverlordClient
ListenableFuture<Void> runTask(String taskId, Object taskObject);
/**
- * Run a "kill" task for a particular datasource and interval. Shortcut to
{@link #runTask(String, Object)}.
- *
- * The kill task deletes all unused segment records from deep storage and
the metadata store. The task runs
- * asynchronously after the API call returns. The resolved future is the ID
of the task, which can be used to
- * monitor its progress through the {@link #taskStatus(String)} API.
- *
- * @param idPrefix Descriptive prefix to include at the start of task IDs
- * @param dataSource Datasource to kill
- * @param interval Interval to kill
- *
- * @return future with task ID
+ * Shortcut to {@link #runKillTask(String, String, Interval, Integer,
DateTime)}.
+ * Note that this method is deprecated and newer implementations must use
{@link #runKillTask(String, String, Interval, Integer, DateTime)}.
*/
+ @Deprecated
Review Comment:
Ah, I was just following the existing pattern to maintain backwards
compatibility to support rolling upgrade/downgrade scenarios and
interface-extension points. I suppose it's ok to remove in this case?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
