fix(sqllab): resultset disappeared on switching tabs

[justinpark]

SUMMARY

Since the rendering refactoring by #20877, the number of redundant rendering reduced.
The FilterableTable re-rendering reduced related to the unnecessary property update but it wasn't reloaded when height is updated. It's mainly caused by functional component refactoring at #21136 which uses useRef to capture the height so it won't refreshed with the updated height prop properly.

https://github.com/EugeneTorap/superset/blob/66b704c13ae5b3840109c584989b1248cafb86e1/superset-frontend/src/components/FilterableTable/index.tsx#L335-L336

This occurs the empty result-set problem due to react-virtualized rendering issue with NaN height value.

This commit fixes the rendering issue with the updated height value.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

• Before

filterableTable--before.mov

• After

filterableTable--after.mov

TESTING INSTRUCTIONS

• Go to SqlLab
• Open 2 tabs in UI
• Run query in 1 and wait for results to show
• Switch to other tab, then back to original tab

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
  cc: @EugeneTorap

[codecov]

Codecov Report

Merging #21741 (fcefb26) into master (0c46149) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #21741      +/-   ##
==========================================
- Coverage   66.84%   66.84%   -0.01%     
==========================================
  Files        1800     1800              
  Lines       68940    68933       -7     
  Branches     7334     7332       -2     
==========================================
- Hits        46084    46079       -5     
+ Misses      20964    20963       -1     
+ Partials     1892     1891       -1     

Flag	Coverage Δ
javascript	53.15% <ø> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
...-frontend/src/components/FilterableTable/index.tsx	72.67% <ø> (+0.04%)	⬆️

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

[EugeneTorap]

Can we merge it?