feat: add name, description and non null tables to RLS

[dpgaspar]

SUMMARY

Add the following changes and improvements to Row Level Security:

• Adds a unique name field (Adds a short name to each RLS, adds context for each RLS)
• Adds a optional description field (adds the possibility to add a detailed description for each RLS)
• Adds a validation that enforces that a new RLS must have an associated dataset (table)

List view

Add view, when adding a new RLS with an already existing name

Add view, validate at least one dataset is associated

cc @altef

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #20432 (fdf5e69) into master (c2f01a6) will decrease coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #20432      +/-   ##
==========================================
- Coverage   66.70%   66.57%   -0.13%     
==========================================
  Files        1739     1739              
  Lines       65153    65166      +13     
  Branches     6899     6899              
==========================================
- Hits        43462    43387      -75     
- Misses      19938    20026      +88     
  Partials     1753     1753              

Flag	Coverage Δ
hive	53.71% <100.00%> (+0.02%)	⬆️
mysql	?
postgres	?
presto	53.57% <100.00%> (+0.02%)	⬆️
python	82.49% <100.00%> (-0.28%)	⬇️
sqlite	82.12% <100.00%> (+<0.01%)	⬆️
unit	50.56% <100.00%> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset/connectors/sqla/models.py	89.99% <100.00%> (-0.35%)	⬇️
superset/connectors/sqla/views.py	91.26% <100.00%> (+0.21%)	⬆️
superset/sql_validators/postgres.py	50.00% <0.00%> (-50.00%)	⬇️
superset/views/database/mixins.py	60.34% <0.00%> (-20.69%)	⬇️
superset/sql_validators/__init__.py	80.00% <0.00%> (-20.00%)	⬇️
superset/databases/commands/update.py	85.71% <0.00%> (-8.17%)	⬇️
superset/common/utils/dataframe_utils.py	85.71% <0.00%> (-7.15%)	⬇️
superset/databases/commands/create.py	86.27% <0.00%> (-5.89%)	⬇️
superset/db_engine_specs/postgres.py	91.52% <0.00%> (-5.09%)	⬇️
superset/databases/api.py	90.67% <0.00%> (-4.83%)	⬇️
... and 15 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c2f01a6...fdf5e69. Read the comment docs.

[betodealmeida]

Maybe?

Suggested change

	rls.name = f"rls{rls.id}"
	rls.name = f"rls_{rls.id}"