[GitHub] [druid] paul-rogers commented on pull request #13766: Operator conversion deny list

["paul-rogers (via GitHub)" <gi...@apache.org>]

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-unsubscribe@druid.apache.org

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


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