[GitHub] [incubator-superset] john-bodley commented on pull request #10130: fix(security): dbs/clusters perm

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

john-bodley commented on pull request #10130:
URL: https://github.com/apache/incubator-superset/pull/10130#issuecomment-647734367


   @bkyryliuk could you elaborate some more in relation to:
   
   > 1 thing would be useful to think about -> automatic FAB permission rename on the DB renames
   
   and 
   
   > 1 more thing would be nice to see in this PR -> keeping FAB with DB permission in sync, e.g. cleaning
   
   If you referring to the possible cascading of renames, I think if we moved completely to hybrid attributes this would be solved.


----------------------------------------------------------------
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: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org