[GitHub] [druid] zachjsh commented on a change in pull request #11720: add optional system schema authorization

[GitBox <gi...@apache.org>]

zachjsh commented on a change in pull request #11720:
URL: https://github.com/apache/druid/pull/11720#discussion_r711897825



##########
File path: integration-tests/docker/environment-configs/common
##########
@@ -72,6 +72,7 @@ druid_coordinator_kill_supervisor_on=true
 druid_coordinator_kill_supervisor_period=PT10S
 druid_coordinator_kill_supervisor_durationToRetain=PT0M
 druid_coordinator_period_metadataStoreManagementPeriod=PT10S
+druid_sql_planner_authorizeSystemTablesDirectly=true

Review comment:
       Seems like it might be wasteful to have integration tests for both true and false, but since the default is disabled here, I think we may still want to have ITs for when this feature is disabled.




-- 
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