You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2022/12/06 11:29:39 UTC

[GitHub] [drill] cgivre commented on pull request #2713: DRILL-8359: Add mount and unmount command support to the filesystem plugin

cgivre commented on PR #2713:
URL: https://github.com/apache/drill/pull/2713#issuecomment-1339182473

   @jnturton 
   I guess my first question is whose permissions will these commands run under?  Are we impersonating the user for them? 
   I do like the idea of adding a boot level option to turn this on and off. 
   
   Another thing to think about is making sure that users can't arbitrarily add this code somehow to a query.
   
   IE:
   ```sql
   SELECT * 
   FROM table(dfs.test.something (mountCommand => 'rm -rf *')
   ```
   
   
   


-- 
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: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org