zachjsh commented on a change in pull request #11501:
URL: https://github.com/apache/druid/pull/11501#discussion_r677870532



##########
File path: 
server/src/main/java/org/apache/druid/client/indexing/HttpIndexingServiceClient.java
##########
@@ -70,7 +70,7 @@ public HttpIndexingServiceClient(
   public void killUnusedSegments(String idPrefix, String dataSource, Interval 
interval)

Review comment:
       I'm not sure what this class is used for since there arent any docs 
about it, but will adding the parameter here instead of defaulting to false 
break anything? If not, any reason not to add?




-- 
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]

Reply via email to