[Bug]: Rapid table and database searches are not debounced
Maintainers usually reply within 2 days
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 25/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Domain
- frontend
Research direction
Start at the dashboard table/database menu and the regression runner that mounts the TablesMenu setup with fake timers. Run the five reported cases and verify that rapid input makes one request for the final keyword, clearing replaces pending work, and unmounting cancels callbacks.
Written by the indexing model from the issue text.
Description
What happened?
Rapid typing schedules one search per input event because each event creates a separate debounce wrapper. Clearing the field leaves earlier callbacks pending, and pending callbacks survive component unmount.
Expected: one request for the final keyword per field after 300ms; clear replaces the pending search; unmount cancels callbacks that have not run.
Affects Versions
master at 061eb724e93592fe87cf44b61af34c1d799f5db6
What table formats are you seeing the problem on?
No response
What engines are you seeing the problem on?
No response
How to reproduce
- Open the dashboard table/database menu.
- Type several characters into either search field within 300ms.
- Inspect the search requests: every pending callback runs instead of only the last.
- Repeat while clearing the field or leaving the view before 300ms. Earlier pending callbacks still run.
Relevant log output
Anything else
The regression runner compiles and mounts the actual TablesMenu setup with fake timers. All five cases fail on the base and pass with per-instance timers and unmount cleanup. A tested fix is ready in hutiefang76/amoro branch fix/table-menu-search-debounce.
Are you willing to submit a PR?
- Yes I am willing to submit a PR!
Code of Conduct
- I agree to follow this project's Code of Conduct
- Dominant language
- Java
- Stars
- 1.2k
- Forks
- 397
- Avg merge
- 1d 21h
- Merged PRs (30d)
- 19
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/amoro
-
[CI] Core CI doesn't run for changes to amoro-format-lance (and amoro-web)Possibly taken @MarkAlex1234 claimed this 4 days ago. Open
Difficulty 1/5 Under an hour Newbie friendliness 88/100
Maintainers usually reply within 2 days
-
fix: low-contrast secondary text color in global theme tokenMay be free again A pull request for this issue was closed without being merged. Openstale
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
apache/amoro#4157 · 1 comment ·
Maintainers usually reply within 2 days
-
fix: mock api server enabled in production build masks real error/loading uxMay be free again A pull request for this issue was closed without being merged. Openstale
Difficulty 1/5 Under an hour Newbie friendliness 78/100
apache/amoro#4155 · 1 comment ·
Maintainers usually reply within 2 days
-
[Bug]: After 0.9, disabling self-optimizing leaves process RUNNING instead of CLOSEDPossibly taken A pull request linked to this issue is open or already merged. Opentype:bug
Difficulty 4/5 3-5 days Newbie friendliness 48/100
Maintainers usually reply within 2 days
-
[Bug]: AMS Spark type rendering loses Iceberg timestamp-without-zone semanticsPossibly taken @hutiefang76 claimed this 2 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
Maintainers usually reply within 2 days
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
sqlcipher/sqlcipher-android#97 · 1 comment ·
-
bug IIIF interoperability
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Netcracker/qubership-integration-platform#1046 ·
Maintainers usually reply within 2 days
-
`check_java_version()` fails when Java path contains spaces (Windows / Git Bash, `C:\Program Files`)Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100