Skip to content

Commit

Permalink
[CLN] Remove QuotaProvider (#3061)
Browse files Browse the repository at this point in the history
## Description of changes
Improvements & Bug fixes
- Remove the `QuotaProvider` abstract class as it is an implementation
choice specific to `QuotaEnforcer` and does not need an exposed hook.

## Test plan
*How are these changes tested?*

- [ ] Tests pass locally with `pytest` for python, `yarn test` for js,
`cargo test` for rust

## Documentation Changes
*Are all docstrings for user-facing APIs updated if required? Do we need
to make documentation changes in the [docs
repository](https://github.com/chroma-core/docs)?*
  • Loading branch information
drewkim authored Nov 4, 2024
1 parent 4dcfd99 commit 2d78a3b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
1 change: 0 additions & 1 deletion chromadb/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@
"chromadb.execution.executor.abstract.Executor": "chroma_executor_impl",
"chromadb.ingest.Consumer": "chroma_consumer_impl",
"chromadb.ingest.Producer": "chroma_producer_impl",
"chromadb.quota.QuotaProvider": "chroma_quota_provider_impl",
"chromadb.quota.QuotaEnforcer": "chroma_quota_enforcer_impl",
"chromadb.rate_limit.RateLimitEnforcer": "chroma_rate_limit_enforcer_impl",
"chromadb.segment.SegmentManager": "chroma_segment_manager_impl",
Expand Down
12 changes: 0 additions & 12 deletions chromadb/quota/__init__.py
Original file line number Diff line number Diff line change
@@ -1,18 +1,6 @@
from abc import abstractmethod
from typing import Callable, TypeVar, Any
from chromadb.config import Component, System

T = TypeVar("T", bound=Callable[..., Any])


class QuotaProvider(Component):
"""
Retrieves quotas for resources within a system.
"""

def __init__(self, system: System) -> None:
super().__init__(system)


class QuotaEnforcer(Component):
"""
Expand Down

0 comments on commit 2d78a3b

Please sign in to comment.