clintropolis commented on a change in pull request #11959:
URL: https://github.com/apache/druid/pull/11959#discussion_r755894673
##########
File path:
sql/src/test/java/org/apache/druid/sql/avatica/DruidAvaticaHandlerTest.java
##########
@@ -1097,9 +1101,24 @@ public void testParameterBinding() throws Exception
}
@Test
- public void testSysTableParameterBinding() throws Exception
+ public void testSysTableParameterBindingRegularUser() throws Exception
{
- PreparedStatement statement = client.prepareStatement("SELECT COUNT(*) AS
cnt FROM sys.servers WHERE servers.host = ?");
+ PreparedStatement statement =
+ client.prepareStatement("SELECT COUNT(*) AS cnt FROM sys.servers WHERE
servers.host = ?");
Review comment:
ah nevermind, found the change in `CalciteTests` that limits to
datasource and view permissions, leaving here in case anyone else wonders the
same thing
--
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]
