-
Notifications
You must be signed in to change notification settings - Fork 3.3k
perf(cosmos): improve pkrange cache memory usage #46297
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
Merged
Merged
Changes from 25 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
8b03fa2
perf(cosmos): share pk range cache + __slots__ + skip .upper()
tvaron3 3ec8f5e
perf(cosmos): add PKRange namedtuple for compact partition key range …
tvaron3 2cd31c6
fix: resolve pylint, mypy, cspell errors in PKRange change
tvaron3 5448e75
perf(cosmos): add __slots__ to _PartitionHealthInfo + comments on Ran…
tvaron3 a63db88
fix: mypy type annotation + move cspell to cosmos package level
tvaron3 5407306
merge: resolve cspell.json conflict with upstream/main
tvaron3 5a0992f
test(cosmos): add integration + fault injection tests for shared cache
tvaron3 e1d4152
fix(cosmos): address review - clear_cache identity, PKRange indexing,…
tvaron3 7f04560
chore: remove harness artifacts from tracked files
tvaron3 44e87b0
fix(cosmos): resolve test failures — PKRange dict equality, test updates
tvaron3 a64fe14
chore: remove stale .temp artifact
tvaron3 e429f92
fix(cosmos): session token parents.copy(), shared cache test isolatio…
tvaron3 fafee80
chore: remove .temp artifact
tvaron3 dd32caf
fix(cosmos): test fixes — PKRange field assertions, remove looping fa…
tvaron3 770c5b1
test(cosmos): add async versions of all shared cache tests
tvaron3 bd830a0
fix(cosmos): async tests — drop enable_cross_partition_query, use que…
tvaron3 b0780c6
fix(cosmos): async tests — populate PK range cache via direct provide…
tvaron3 506d3fe
fix(cosmos): address iter-2 review — shared locks, cache release, PKR…
tvaron3 2320aae
fix(cosmos): pylint docstrings on _resolve_endpoint + async CRUD test…
tvaron3 dffcc7b
chore: untrack .coding-harness/ harness artifacts
tvaron3 4aaa556
ci: retrigger pipelines (flaky test_health_check_failure_startup_asyn…
tvaron3 ce63b96
doc(cosmos): document PKRange.__contains__ truthy-presence semantics
tvaron3 19e046e
chore: untrack .coding-harness/ harness artifacts (proper gitignore)
tvaron3 c474821
test(cosmos): bump test_timeout_for_read_items delay 2s→3s
tvaron3 96e1323
Merge remote-tracking branch 'upstream/main' into fix/shared-pk-range…
tvaron3 014dc89
chore(cosmos): address PR review comments
tvaron3 7a0730c
docs(cosmos): explain shared routing-map cache module-level globals
tvaron3 220fcf0
fix(cosmos): scope async pk-range locks per event loop, reset cache b…
Copilot 335f6f5
Fix shared-cache test fixture to preserve dict identity
tvaron3 e2c698c
F2: preserve per-collection locks across clear_cache to keep single-f…
tvaron3 db156df
Address xinlian12 review + fix test_multi_client_shared_cache_queries
tvaron3 28fe446
Merge branch 'main' into fix/strip-pk-range-fields
tvaron3 2f70bbf
Fix Build Analyze: pylint C4732/C4739 + cspell TOCTOU
2c7a318
Address xinlian Apr 24 review: sync clear_cache + retain status/throu…
tvaron3 1abb340
Revert .gitignore changes — keep PR diff scoped to PKR cache work
tvaron3 eab73eb
Address xinlian review: dedupe _resolve_endpoint + PKRange construction
tvaron3 ea36af0
fix: pylint C4740 — restore type annotation on _resolve_endpoint
tvaron3 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.