Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Swaps search optimizations #6384

Draft
wants to merge 65 commits into
base: develop
Choose a base branch
from
Draft

Swaps search optimizations #6384

wants to merge 65 commits into from
+664 −350

Conversation

christianbaroni
Copy link
Member

@christianbaroni christianbaroni commented Jan 6, 2025 •
edited
Loading

What changed (plus any additional context for devs)

  • WIP migration to createQueryStore

Screen recordings / screenshots

What to test

@christianbaroni
Add createRemoteRainbowStore
6dd30ee
@christianbaroni
Fix import path
68101ed
@christianbaroni
Improve types, rename to createRainbowQueryStore
8a48827
@christianbaroni
Clean up createRainbowStore
c8121cb
@christianbaroni
Ensure internal state is persisted when partialize is used, remove an…
3fa324f 
…y types
@christianbaroni
Make createBlendedPartialize faster, improve typing
938c7de
@christianbaroni
Improve persist map readability
4a6fc53
@christianbaroni
Add dynamic params, API cleanup
d2c0061
@christianbaroni
Misc. cleanup
eef9903
@christianbaroni
Add time utility
96d8fd4
@christianbaroni
Guard console.log
0b59554
@christianbaroni
Merge remote-tracking branch 'origin/develop' into @christian/remote-…
9966fa2 
…rainbow-store
@christianbaroni
Add setData, clean up caching and staleTime logic, rename to createQu…
c02515d 
…eryStore
@christianbaroni
Use consistent store naming
a8d4c73
@christianbaroni
Improve error handling, add getStatus(), reduce type redundancy, omit…
7dbb6ef 
… store methods from persisted state
@christianbaroni
[createRainbowStore] Support maps and sets internally
2f86899
@christianbaroni
Allow dynamic params for internal state: ($, store) => $(store).state
791757e
@christianbaroni
[createRainbowStore] Catch up with develop
11f8fdf
@christianbaroni
Merge remote-tracking branch 'origin/develop' into @christian/remote-…
5d9d5c4 
…rainbow-store
@christianbaroni
Remove unnecessary assignment
61e97f4
@christianbaroni
Minor docs cleanup
fda20e2
@christianbaroni
Split types to better obscure private internal state
96a12a7
@christianbaroni
Fix cache overwriting, expose and track queryKey, improve setData d…
a1d1b45 
…ocs, add overloads, misc. fixes
@christianbaroni
Fix signal logging
8b11f97
@christianbaroni
Merge remote-tracking branch 'origin/develop' into @christian/remote-…
b8c4a3c 
…rainbow-store
@christianbaroni
Organize types, skip string conversion if unneeded in getOrCreateAtta…
c677ad2 
…chValue
@christianbaroni
Merge branch 'develop' into @christian/remote-rainbow-store
b969de5
@christianbaroni
Add AbortController, abortInterruptedFetches setting (default true)
c898f97
@christianbaroni
Ensure effectiveParams and currentQueryKey are always in sync within …
a6f8998 
…each fetch operation
@christianbaroni
Improve logging, fix error handling, improve param equality check acc…
055cdd4 
…uracy, make fetch return data, expose params to transform, misc. optimizations
@christianbaroni
[createRainbowStore] Fix debounced persistence, call partialize onl…
8530bd1 
…y within `lazyPersist`, handle maps and sets internally if neither `deserializer` nor `serializer` are provided, add `persistThrottleMs` to config
@christianbaroni
Remove commented code
07e1c23
@christianbaroni
Clean up logs, reduce unnecessary subscription work
bd63de7
@christianbaroni
[createRainbowStore] Adjust default persistThrottleMs
958cd90
@christianbaroni christianbaroni force-pushed the @christian/remote-rainbow-store branch from 7dba53b to 958cd90 Compare January 10, 2025 01:05
@christianbaroni christianbaroni force-pushed the @christian/query-store-token-search branch from c7e4a0a to 0f96d10 Compare January 10, 2025 01:50
@christianbaroni christianbaroni force-pushed the @christian/query-store-token-search branch from 0f96d10 to 90a45cb Compare January 10, 2025 02:09
Base automatically changed from @christian/remote-rainbow-store to develop January 10, 2025 10:43
@brunobar79 brunobar79 added release for release blockers and release candidate branches and removed release for release blockers and release candidate branches labels Jan 23, 2025
@brunobar79 brunobar79 added the release for release blockers and release candidate branches label Jan 31, 2025
@jinchung jinchung removed the release for release blockers and release candidate branches label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
performance performance related improvements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@christianbaroni @brunobar79 @jinchung