[GitHub] [druid] suneet-s commented on pull request #10765: Add the ability to supply client certificate to dsql command line tool.

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

suneet-s commented on pull request #10765:
URL: https://github.com/apache/druid/pull/10765#issuecomment-765574168


   @misqos Thanks for the contribution! I've re-triggered the failed test because I think it's flaky.
   
   Do you think we should add documentation for this in https://github.com/apache/druid/blob/master/docs/tutorials/tutorial-query.md#query-sql-via-dsql ?
   
   Also, do you think we can add an automated test for this by reusing the current integration test cluster that's set up for the security integration tests? See the tests under the group `TestNGGroup.SECURITY` and the cluster set up in https://github.com/apache/druid/blob/master/integration-tests/docker/docker-compose.security.yml


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

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