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.

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

Vec 283 Add the ability to Insert,Update,Upsert,Query #10

Merged
merged 20 commits into from
Aug 27, 2024
Merged

Conversation

jdogmcsteezy
Copy link

Major changes/improvements:

  1. Combined the Admin and regular client. (Effectively renaming AdminClient to Client)
  2. Added ci/cd test infrastructure. I will need to play catch up on est coverage for existing fetaures.
  3. Added Insert
  4. Added Update
  5. Added Upsert
  6. Added VectorSearch
  7. Refactored the ChannelProvider to return a new Connection struct. This struct combines the various grpc clients so they do not need to be repeatedly created. This follows the grpc best practices.

@jdogmcsteezy jdogmcsteezy requested a review from reugn August 23, 2024 22:48
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

client.go Outdated Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
Copy link
Author

@jdogmcsteezy jdogmcsteezy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made most of the requested changes

client.go Outdated Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Outdated Show resolved Hide resolved
client.go Show resolved Hide resolved
@jdogmcsteezy jdogmcsteezy requested a review from reugn August 26, 2024 22:45
@jdogmcsteezy jdogmcsteezy merged commit b7ab11d into main Aug 27, 2024
4 checks passed
@jdogmcsteezy jdogmcsteezy deleted the VEC-283 branch August 27, 2024 16:01
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.

3 participants