refactor: Remove obsolete HiveEngineSpec.fetch_logs method

[john-bodley]

SUMMARY

PyHive now contains the fetch_log method which makes the HiveEngineSpec.fetch_logs() method obsolete—per the comment it mentioned it should be contributed back to PyHive.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

CI.

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 #20631 (59437ea) into master (429f246) will increase coverage by 0.80%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##           master   #20631      +/-   ##
==========================================
+ Coverage   65.76%   66.56%   +0.80%     
==========================================
  Files        1813     1791      -22     
  Lines       69437    68609     -828     
  Branches     7450     7318     -132     
==========================================
+ Hits        45667    45672       +5     
+ Misses      21850    21044     -806     
+ Partials     1920     1893      -27     

Flag	Coverage Δ
hive	52.93% <0.00%> (+0.06%)	⬆️
javascript	52.75% <0.00%> (-0.74%)	⬇️
mysql	78.08% <0.00%> (-0.31%)	⬇️
postgres	78.14% <0.00%> (-0.30%)	⬇️
presto	52.83% <0.00%> (+0.06%)	⬆️
python	81.30% <0.00%> (+2.38%)	⬆️
sqlite	76.64% <0.00%> (-0.27%)	⬇️
unit	50.96% <0.00%> (?)

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

Impacted Files	Coverage Δ
...-chart-table/src/DataTable/utils/useMountedMemo.ts	0.00% <0.00%> (-87.50%)	⬇️
...lUtils/getControlValuesCompatibleWithDatasource.ts	7.69% <0.00%> (-73.79%)	⬇️
...trols/DateFilterControl/components/CommonFrame.tsx	0.00% <0.00%> (-57.15%)	⬇️
superset-frontend/src/utils/getDatasourceUid.ts	50.00% <0.00%> (-50.00%)	⬇️
...ugin-chart-table/src/DataTable/hooks/useSticky.tsx	4.21% <0.00%> (-47.37%)	⬇️
...s/CRUD/data/dataset/AddDataset/LeftPanel/index.tsx	46.42% <0.00%> (-38.19%)	⬇️
...chart-controls/src/shared-controls/dndControls.tsx	28.30% <0.00%> (-30.04%)	⬇️
...d/src/views/CRUD/data/dataset/AddDataset/index.tsx	33.33% <0.00%> (-24.57%)	⬇️
superset-frontend/src/components/Select/styles.tsx	81.13% <0.00%> (-18.87%)	⬇️
...perset-frontend/src/views/components/RightMenu.tsx	60.83% <0.00%> (-17.64%)	⬇️
... and 404 more

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

[bkyryliuk]

Thanks for the cleanup !

[michael-s-molina]

LGTM