Skip to content

feat: implement LoadGuard for enhanced resource management in Caching… - #3

Open
liliu-z wants to merge 1 commit into
masterfrom
cachecs
Open

feat: implement LoadGuard for enhanced resource management in Caching…#3
liliu-z wants to merge 1 commit into
masterfrom
cachecs

Conversation

@liliu-z

@liliu-z liliu-z commented Jun 25, 2025

Copy link
Copy Markdown
Owner

No description provided.

…Layer

- Add SegmentLoadGuard with dual-level admission control
- Implement Cell-based resource management for fine-grained control
- Support evictable/non-evictable data classification
- Add memory monitor for physical resource tracking
- Integrate with existing segment loader with backward compatibility
- Add comprehensive tests and documentation

This addresses resource management issues introduced by CachingLayer
by providing:
- Shallow load admission (segment level)
- Deep load admission (cell level)
- Cache ratio configuration for hot data protection
- Logical and physical watermark protection
- Eviction integration support
liliu-z pushed a commit that referenced this pull request Mar 23, 2026
## Summary
- Add `TestMilvusClientRbacPrefixIsolation` class to
`test_milvus_client_rbac.py`
- Validates RBAC metadata integrity and permission isolation when
usernames/roles have prefix relationships (e.g. `user1` vs `user11`),
covering issue milvus-io#47998 and PR milvus-io#48053
- Single test method with setup/teardown per run, using random prefix
for test data isolation

## Related Issues
issue: milvus-io#47998
pr: milvus-io#48053

## Test Report
https://zilliverse.feishu.cn/docx/HaFld6sCyoMz19xOsC3cPCqdncg

## Test plan
- [x] Verified on v2.6.9 (pre-fix): FAIL on #3 DescribeUser
(empty-string role) and milvus-io#23 ListDatabases
- [x] Verified on master-20260319 (post-fix): all PASS
- [x] Teardown cleans up all test data using `force_drop`, no residual
data

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Signed-off-by: lyyyuna <yiyang.li@zilliz.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant