paul-rogers commented on PR #13766: URL: https://github.com/apache/druid/pull/13766#issuecomment-1426105902
Just for the record, the goal of this PR is to block input sources not supported in a given cluster. The _right_ way to do this is to create a deny list for input sources, and block them in all the places we use them. Blocking SQL leaves a hole for native ingest. It leaves a hole if an extension provides a table function that uses a disallowed input source, etc. Blocking at the site of use allows a better error message: "the Druid input source foo is not supported in this installation." -- 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
