mluvin-stripe commented on code in PR #17579:
URL: https://github.com/apache/pinot/pull/17579#discussion_r2743132180
##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentUploadDownloadRestletResource.java:
##########
@@ -156,6 +161,9 @@ public class PinotSegmentUploadDownloadRestletResource {
@Inject
AccessControlFactory _accessControlFactory;
+ @Inject
+ DiskUtilizationChecker _diskUtilizationChecker;
Review Comment:
I described in more detail why I went with this approach in
https://github.com/apache/pinot/issues/17557
we'd like a way to enable/disable each resource utilization checker
individually, instead of being automatically opted in when more are added in
the future
--
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]
