chore: refactor FilterableTable to functional component

[EugeneTorap]

SUMMARY

Converted FilterableTable class component into functional component.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

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
  @michael-s-molina @kgabryje @lyndsiWilliams @villebro

[codecov]

Codecov Report

Merging #21136 (41c14b9) into master (642a0d7) will increase coverage by 0.06%.
The diff coverage is 74.28%.

❗ Current head 41c14b9 differs from pull request most recent head 66b704c. Consider uploading reports for the commit 66b704c to get more accurate results

@@            Coverage Diff             @@
##           master   #21136      +/-   ##
==========================================
+ Coverage   66.26%   66.32%   +0.06%     
==========================================
  Files        1772     1772              
  Lines       67649    67659      +10     
  Branches     7205     7212       +7     
==========================================
+ Hits        44825    44877      +52     
+ Misses      20981    20939      -42     
  Partials     1843     1843              

Flag	Coverage Δ
javascript	52.10% <76.47%> (+0.01%)	⬆️

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

Impacted Files	Coverage Δ
superset/models/helpers.py	39.31% <0.00%> (-0.09%)	⬇️
...-frontend/src/components/FilterableTable/index.tsx	72.34% <76.47%> (+2.34%)	⬆️
superset/models/core.py	88.75% <0.00%> (+0.23%)	⬆️
superset/connectors/sqla/models.py	90.32% <0.00%> (+0.52%)	⬆️
superset/models/sql_lab.py	77.01% <0.00%> (+2.01%)	⬆️
superset/db_engine_specs/presto.py	87.97% <0.00%> (+6.01%)	⬆️

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

[zhaoyongjie]

/testenv up

[github-actions]

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

[zhaoyongjie]

LGTM! Thanks for the improvement!

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.

[kgabryje]

I think we should wrap the export in React.memo to replicate the behaviour of PureComponent from class component