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

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

bkyryliuk commented on pull request #10130:
URL: https://github.com/apache/incubator-superset/pull/10130#issuecomment-648976056


   @john-bodley actually scratch that, looks like it is already happening in the sync. I've seen some abandoned permissions, but there is probably a difference reason for them.
   
   
   > @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'm not sure if the database perm relates to the datasource per), 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