fix(native-filters): Overhead when changing the filter name

[michael-s-molina]

SUMMARY

Fixes the overhead when changing the filter name by using a debounced function.

@junlincc @jinghua-qa

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2021-09-28.at.3.44.10.PM.mov

Screen.Recording.2021-09-28.at.3.42.24.PM.mov

TESTING INSTRUCTIONS

Check the videos for 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

[codecov]

Codecov Report

Merging #16877 (4c81c99) into master (b35645c) will decrease coverage by 0.00%.
The diff coverage is 88.88%.

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

@@            Coverage Diff             @@
##           master   #16877      +/-   ##
==========================================
- Coverage   77.05%   77.04%   -0.01%     
==========================================
  Files        1021     1021              
  Lines       54693    54695       +2     
  Branches     7457     7457              
==========================================
  Hits        42141    42141              
- Misses      12307    12309       +2     
  Partials      245      245              

Flag	Coverage Δ
javascript	71.19% <88.88%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
...eFilters/FiltersConfigModal/FiltersConfigModal.tsx	92.22% <88.88%> (-2.10%)	⬇️

---

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 b35645c...4784c4f. Read the comment docs.

[villebro]

LGTM. Do we have guidelines for when to use SLOW_DEBOUNCE vs FAST_DEBOUNCE? I've also noticed we're using custom debounces all around the codebase. Some locally defined debounces that I found that should maybe be harmonized/centralized: 10, 100, 200, 250, 300, 500, 600, 1000, 2000 (maybe something for the style guidelines).

[geido]

/testenv up FEATURE_DASHBOARD_NATIVE_FILTERS=true FEATURE_DASHBOARD_CROSS_FILTERS=true FEATURE_DASHBOARD_NATIVE_FILTERS_SET=true FEATURE_DASHBOARD_FILTERS_EXPERIMENTAL=true

[github-actions]

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

[geido]

LGTM! Thanks for the improvement!

[michael-s-molina]

, 100, 200, 250, 300, 500, 600, 1000, 2000 (maybe something for the style guide

That's a great point. I don't think we have a guideline yet.

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.