-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve in-mem typeahead performance #1353
Labels
Comments
junaidzm13
changed the title
Improve typeahead performance
Improve in-mem typeahead performance
May 21, 2024
SVC-GL-OS-SYNC
pushed a commit
to UBS-IB/vuu
that referenced
this issue
May 23, 2024
SVC-GL-OS-SYNC
pushed a commit
to UBS-IB/vuu
that referenced
this issue
May 23, 2024
SVC-GL-OS-SYNC
pushed a commit
to UBS-IB/vuu
that referenced
this issue
May 23, 2024
- the plan is to add a separate test suite for benchmark tests as they are usually more expensive to run.
SVC-GL-OS-SYNC
pushed a commit
to UBS-IB/vuu
that referenced
this issue
May 23, 2024
…urned are unique. Adding test for when some of the values are null
junaidzm13
added a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
junaidzm13
added a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
junaidzm13
added a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
- the plan is to add a separate test suite for benchmark tests as they are usually more expensive to run.
junaidzm13
pushed a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
…urned are unique. Adding test for when some of the values are null
junaidzm13
added a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
junaidzm13
added a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
junaidzm13
added a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
- the plan is to add a separate test suite for benchmark tests as they are usually more expensive to run.
junaidzm13
pushed a commit
to junaidzm13/vuu
that referenced
this issue
May 23, 2024
…urned are unique. Adding test for when some of the values are null
heswell
added a commit
that referenced
this issue
May 23, 2024
…ormance #1353 improve in-mem typeahead's performance
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We're currently sorting all table rows only to get 10 distinct values in the end on every typeahead RPC call. Which turns out to be a bottleneck if we have a large table. Local tests have shown that it takes around 1 second for the method to return 10 distinct values for a table with 1 million rows.
It might be better if we can instead return the first 10 distinct values that match the given criteria / filters and not care about sorting (since we're returning only 10 values, sorted / unsorted wouldn't make much of a diff in terms of UX since the user doesn't have the scroll functionality anyways).
The text was updated successfully, but these errors were encountered: