Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Modify ChromaCloud provider to check base limitations of chroma cloud before upsert to prevent 422#4705

Merged
timothycarambat merged 2 commits intomasterfrom
4704-chroma-cloud-quota-apply
Dec 3, 2025
Merged

Modify ChromaCloud provider to check base limitations of chroma cloud before upsert to prevent 422#4705
timothycarambat merged 2 commits intomasterfrom
4704-chroma-cloud-quota-apply

Conversation

@timothycarambat
Copy link
Member

@timothycarambat timothycarambat commented Dec 3, 2025

Pull Request Type

  • ✨ feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

resolves #4704

What is in this change?

  • On ChromaCloud vector DB apply base quota limits
  • Duplicate methods on base Chroma provider with no meaningful change to ops since self-hosted don't have these limits
  • Modify the response API to ignore result of add/delete. new API throws on failure, which will be caught at higher level anyway.

Additional Information

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@timothycarambat timothycarambat merged commit b54ac2d into master Dec 3, 2025
1 check passed
@timothycarambat timothycarambat deleted the 4704-chroma-cloud-quota-apply branch December 3, 2025 18:41
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.

[BUG]: Chroma Cloud upsert limitations

1 participant