chore: remove QueryView (ModelView)

[mistercrunch]

SUMMARY

This is not needed anymore as we have the "Query Search" page, and the /api/v1/query REST endpoints.

[willbarrett]

Some odd test failures on this one. I'd change deprecate to remove in the PR title - this isn't deprecation, which implies prior notice of removal. This is deleting API endpoints.

@john-bodley @etr2460 @betodealmeida @bkyryliuk are these endpoints in use by your organizations? Trying to cast a wide net early...

[etr2460]

Could you share an example of the URLs that are removed?

[bkyryliuk]

Some odd test failures on this one. I'd change deprecate to remove in the PR title - this isn't deprecation, which implies prior notice of removal. This is deleting API endpoints.

@john-bodley @etr2460 @betodealmeida @bkyryliuk are these endpoints in use by your organizations? Trying to cast a wide net early...

we don't use those, thanks for the cleanup!

[bkyryliuk]

Some odd test failures on this one. I'd change deprecate to remove in the PR title - this isn't deprecation, which implies prior notice of removal. This is deleting API endpoints.

@john-bodley @etr2460 @betodealmeida @bkyryliuk are these endpoints in use by your organizations? Trying to cast a wide net early...

we don't use those, thanks for the cleanup!

[codecov-commenter]

Codecov Report

Merging #10162 into master will increase coverage by 1.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10162      +/-   ##
==========================================
+ Coverage   70.63%   71.69%   +1.06%     
==========================================
  Files         601      405     -196     
  Lines       32286    13312   -18974     
  Branches     3275     3275              
==========================================
- Hits        22804     9544   -13260     
+ Misses       9376     3662    -5714     
  Partials      106      106              

Flag	Coverage Δ
#cypress	55.63% <ø> (+0.02%)	⬆️
#javascript	59.61% <ø> (ø)
#python	?

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

Impacted Files	Coverage Δ
...rontend/src/SqlLab/components/QueryAutoRefresh.jsx	65.90% <0.00%> (-6.82%)	⬇️
superset-frontend/src/SqlLab/actions/sqlLab.js	62.60% <0.00%> (ø)
superset/views/database/filters.py
superset/models/helpers.py
superset/dashboards/commands/exceptions.py
superset/migrations/env.py
superset/utils/cache.py
superset/utils/import_datasource.py
superset/views/utils.py
superset/utils/core.py
... and 188 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 73797b8...72c539c. Read the comment docs.

[mistercrunch]

@etr2460 this removes /queryview/read

Can I get a stamp?