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