clintropolis commented on code in PR #16758:
URL: https://github.com/apache/druid/pull/16758#discussion_r1684202540


##########
indexing-service/src/main/java/org/apache/druid/indexing/common/task/Task.java:
##########
@@ -152,8 +152,7 @@ default int getPriority()
    * the task does not use any. Users can be given permission to access 
particular types of
    * input sources but not others, using the
    * {@link 
org.apache.druid.server.security.AuthConfig#enableInputSourceSecurity} config.
-   * @throws UnsupportedOperationException if the given task type does not 
suppoert input source based security. Such
-   * would be the case, if the task uses firehose.
+   * @throws UnsupportedOperationException if the given task type does not 
suppoert input source based security

Review Comment:
   the default implementation still throws that exception, so i suppose if 
something doesn't implement this method it would happen



-- 
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: commits-unsubscr...@druid.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org

Reply via email to