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. What do you think?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
