fix: properly keep state on queryEditorSetSql on tabstateview PUT

[hughhhh]

SUMMARY

Pass latestQueryId when user updates SQL. Anytime we call the put the update the sql for the current tab, the clientid/latestQueryId changes which cause the sql_json endpoint to fail since we can’t the proper id

The following error is thrown whenever the query id is overwritten and cannot be found.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

• Has associated issue:
• 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 #14579 (bc540ac) into master (0fe2bf1) will decrease coverage by 0.23%.
The diff coverage is 100.00%.

❗ Current head bc540ac differs from pull request most recent head 4e44fab. Consider uploading reports for the commit 4e44fab to get more accurate results

@@            Coverage Diff             @@
##           master   #14579      +/-   ##
==========================================
- Coverage   77.40%   77.16%   -0.24%     
==========================================
  Files         958      958              
  Lines       48326    48327       +1     
  Branches     5677     5677              
==========================================
- Hits        37405    37291     -114     
- Misses      10721    10836     +115     
  Partials      200      200              

Flag	Coverage Δ
hive	?
javascript	72.47% <100.00%> (+<0.01%)	⬆️
presto	?

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

Impacted Files	Coverage Δ
superset-frontend/src/SqlLab/actions/sqlLab.js	58.97% <ø> (ø)
...rontend/src/explore/components/optionRenderers.tsx	100.00% <ø> (ø)
superset/examples/country_map.py	26.31% <ø> (ø)
superset-frontend/src/components/Badge/index.tsx	100.00% <100.00%> (ø)
.../components/Header/HeaderActionsDropdown/index.jsx	68.42% <100.00%> (ø)
superset/db_engines/hive.py	0.00% <0.00%> (-82.15%)	⬇️
superset/db_engine_specs/hive.py	70.32% <0.00%> (-17.08%)	⬇️
superset/db_engine_specs/presto.py	83.36% <0.00%> (-6.53%)	⬇️
superset/views/database/mixins.py	81.03% <0.00%> (-1.73%)	⬇️
superset/connectors/sqla/models.py	88.37% <0.00%> (-1.70%)	⬇️
... and 4 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 0fe2bf1...4e44fab. Read the comment docs.

[hughhhh]

/testenv up

[hughhhh]

/testenv up

[betodealmeida]

[github-actions]

@hughhhh Ephemeral environment spinning up at http://54.201.154.143:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[github-actions]

@hughhhh Ephemeral environment spinning up at http://35.167.148.245:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.