chore: Migrate /superset/tables/* to API v1

[diegomedina248]

SUMMARY

Continuing the effort on deprecating all /superset/ REST API endpoints
Deprecates /tables/<int:db_id>/<schema>/
Deprecates /tables/<int:db_id>/<schema>/<force_refresh>/

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 #22501 (3eb0665) into master (e18bf3f) will increase coverage by 0.04%.
The diff coverage is 97.01%.

@@            Coverage Diff             @@
##           master   #22501      +/-   ##
==========================================
+ Coverage   67.30%   67.35%   +0.04%     
==========================================
  Files        1876     1877       +1     
  Lines       72086    72135      +49     
  Branches     7868     7869       +1     
==========================================
+ Hits        48521    48583      +62     
+ Misses      21548    21534      -14     
- Partials     2017     2018       +1     

Flag	Coverage Δ
hive	52.72% <52.38%> (-0.01%)	⬇️
mysql	78.74% <96.82%> (?)
postgres	78.78% <98.41%> (+0.02%)	⬆️
presto	52.62% <52.38%> (-0.01%)	⬇️
python	82.00% <98.41%> (+0.06%)	⬆️
sqlite	77.12% <98.41%> (+0.04%)	⬆️
unit	51.67% <52.38%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
superset/constants.py	100.00% <ø> (ø)
superset-frontend/src/hooks/apiResources/tables.ts	80.00% <50.00%> (ø)
superset/databases/api.py	91.49% <95.00%> (+0.19%)	⬆️
...s/CRUD/data/dataset/AddDataset/LeftPanel/index.tsx	84.61% <100.00%> (ø)
superset/databases/commands/exceptions.py	97.05% <100.00%> (+0.13%)	⬆️
superset/databases/commands/tables.py	100.00% <100.00%> (ø)
superset/databases/schemas.py	98.37% <100.00%> (+0.02%)	⬆️
superset/views/core.py	74.64% <100.00%> (+0.24%)	⬆️
.../src/utils/getChartRequiredFieldsMissingMessage.ts	66.66% <0.00%> (-13.34%)	⬇️
superset/reports/commands/log_prune.py	85.71% <0.00%> (-3.58%)	⬇️
... and 36 more

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

[dpgaspar]

Looks good!

[dpgaspar]

isn't this:

type QueryData = {
  json: {
    count: number;
    result: {
      options: Table[];
    };
  };

[dpgaspar]

bring the count out of the result object`, like:

{
    count: 10,
    result: [
          {
                 ....
          },
          {
                ....
          },
          ...
    ]
}

[dpgaspar]

One last comment :)

[dpgaspar]

LGTM, after fixing conflicts

[geido]

Tests LGTM!