chore: use contextlib.surpress instead of passing on error

[sebastianliebscher]

SUMMARY

This PR uses contextlib's suppress for passing a specific error, instead of passing in an exception handler. The context manager shortens the code a bit and explicitly states the authors intend to ignore specific errors, thus improving readability.

Also, if I found the possibility, I used the walrus (:=) operator.

TESTING INSTRUCTIONS

run unit and integration tests

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 #24896 (a9edbb5) into master (7397ab3) will increase coverage by 0.06%.
The diff coverage is 71.15%.

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

@@            Coverage Diff             @@
##           master   #24896      +/-   ##
==========================================
+ Coverage   68.96%   69.02%   +0.06%     
==========================================
  Files        1906     1906              
  Lines       74122    74075      -47     
  Branches     8208     8208              
==========================================
+ Hits        51116    51132      +16     
+ Misses      20883    20820      -63     
  Partials     2123     2123              

Flag	Coverage Δ
hive	54.24% <44.23%> (+0.07%)	⬆️
mysql	79.29% <67.30%> (+0.07%)	⬆️
postgres	79.40% <67.30%> (?)
presto	54.14% <46.15%> (+0.07%)	⬆️
python	83.45% <71.15%> (+0.15%)	⬆️
sqlite	77.97% <67.30%> (+0.08%)	⬆️
unit	55.13% <42.30%> (+0.09%)	⬆️

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

Files Changed	Coverage Δ
superset/common/tags.py	0.00% <0.00%> (ø)
superset/db_engine_specs/trino.py	86.66% <33.33%> (+2.92%)	⬆️
superset/db_engine_specs/bigquery.py	71.06% <50.00%> (+0.62%)	⬆️
superset/views/core.py	70.69% <54.54%> (+1.23%)	⬆️
superset/db_engine_specs/presto.py	88.75% <66.66%> (+0.75%)	⬆️
superset/charts/data/api.py	88.69% <100.00%> (-0.27%)	⬇️
superset/db_engine_specs/gsheets.py	77.94% <100.00%> (+1.29%)	⬆️
superset/db_engine_specs/mysql.py	98.83% <100.00%> (-0.02%)	⬇️
superset/db_engine_specs/ocient.py	53.27% <100.00%> (+0.43%)	⬆️
superset/explore/commands/get.py	90.32% <100.00%> (-0.11%)	⬇️
... and 6 more

... and 9 files with indirect coverage changes

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

[john-bodley]

Thanks @sebastianliebscher for the refactor and code cleanup.