Jackie-Jiang commented on code in PR #17579:
URL: https://github.com/apache/pinot/pull/17579#discussion_r2743183602
##########
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:
You can decide which checker to include when initializing the controller,
but all the checkers should be applied. This is the only way to make resource
checkers pluggable. In OSS code, it might not have access to the custom checkers
--
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]
