Closed clrcrl closed 6 years ago
@clrcrl this is awesome! Adding @cmcarthur to take a look as well, but LGTM!
@clrcrl one comment, but otherwise LGTM!
@clrcrl thanks for making that change! Is this ready to be merged? Just tried it out on our Redshift and it works like a charm :)
@drewbanin - I believe it is! I've tested it across two clusters and both worked for me.
Created some views to help admins see which users have permissions on tables/views, and on schemas.
Also made some small tweaks to existing code (deleted a duplicate/unused base model, added a new column to table_stats)