chore: deprecate /superset/filter/... endpoint, migrate to apiv1

[jfrag1]

SUMMARY

Deprecates /superset/filter/<datasource_type>/<datasource_id>/<column_name>/ endpoint

Creates a new, more REST-ful endpoint to replace it

• /api/v1/datasource/<datasource_type>/<datasource_id>/column/<column_name>/values/

Other improvements include better test coverage, better error handling, and an openapi spec

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 #22882 (8c99669) into master (d00ba15) will decrease coverage by 0.17%.
The diff coverage is 97.67%.

❗ Current head 8c99669 differs from pull request most recent head 5ac29ca. Consider uploading reports for the commit 5ac29ca to get more accurate results

@@            Coverage Diff             @@
##           master   #22882      +/-   ##
==========================================
- Coverage   67.19%   67.02%   -0.17%     
==========================================
  Files        1871     1872       +1     
  Lines       71774    71813      +39     
  Branches     7869     7869              
==========================================
- Hits        48230    48135      -95     
- Misses      21526    21660     +134     
  Partials     2018     2018              

Flag	Coverage Δ
hive	?
mysql	78.53% <100.00%> (+0.02%)	⬆️
postgres	?
presto	?
python	81.43% <100.00%> (-0.39%)	⬇️
sqlite	76.91% <100.00%> (+0.02%)	⬆️
unit	51.57% <57.14%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
...l/AdhocFilterEditPopoverSimpleTabContent/index.tsx	62.73% <0.00%> (ø)
superset/datasource/api.py	100.00% <100.00%> (ø)
superset/initialization/__init__.py	91.88% <100.00%> (+0.05%)	⬆️
superset/security/manager.py	95.77% <100.00%> (ø)
superset/views/core.py	75.17% <100.00%> (+0.01%)	⬆️
superset/db_engines/hive.py	0.00% <0.00%> (-85.19%)	⬇️
superset/sql_validators/postgres.py	50.00% <0.00%> (-50.00%)	⬇️
superset/sql_validators/__init__.py	80.00% <0.00%> (-20.00%)	⬇️
superset/db_engine_specs/hive.py	71.64% <0.00%> (-16.05%)	⬇️
superset/db_engine_specs/presto.py	81.09% <0.00%> (-6.94%)	⬇️
... and 15 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[EugeneTorap]

Hi! Should we use dataset name instead of datasource in the new v1 API?

[dpgaspar]

LGTM, just a couple of details

[jfrag1]

Hi! Should we use dataset name instead of datasource in the new v1 API?

Hi there! I think it makes sense to have a /datasource blueprint/namespace in the v1 API since datasets are a type of datasource, but there can be other types as well.

There are several "datasource APIs" in this file that I think can be moved here as well for 3.0.

It is somewhat confusing, but I think it would be more confusing to put all these under /dataset

[geido]

Cypress change looks good