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]