clintropolis commented on a change in pull request #11720:
URL: https://github.com/apache/druid/pull/11720#discussion_r712422050
##########
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:
I considered this, I think it is ok to only test enabled because it just
layers additional checks on top of the disabled code path, so with the newly
added user we should still be covering all of the code paths that were
previously tested, just distributed a bit differently.
--
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]
