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

Add deletion strategy option when creating classes #88

Merged
merged 2 commits into from
Nov 7, 2024

Conversation

rlmanrique
Copy link
Collaborator

@rlmanrique rlmanrique commented Nov 6, 2024

Add the option to set the deletion strategy when creating a collection.

@rlmanrique rlmanrique force-pushed the rodrigo/delete_on_conflict branch 2 times, most recently from ab59303 to 25f43e5 Compare November 6, 2024 12:04
@rlmanrique rlmanrique force-pushed the rodrigo/delete_on_conflict branch from 25f43e5 to b6d8cd8 Compare November 6, 2024 12:29
@rlmanrique rlmanrique requested a review from jfrancoa November 6, 2024 14:40
@rlmanrique
Copy link
Collaborator Author

Ready for review and merge

@jfrancoa
Copy link
Collaborator

jfrancoa commented Nov 6, 2024

If I recall correctly I remember Jero saying at the beginning that this option could be enabled when updating a class, could you also add it into the update_collection logic?

@rlmanrique
Copy link
Collaborator Author

If I recall correctly I remember Jero saying at the beginning that this option could be enabled when updating a class, could you also add it into the update_collection logic?

Oh, I didn't know about it. Let me try it out!

@rlmanrique rlmanrique force-pushed the rodrigo/delete_on_conflict branch 2 times, most recently from c1f832c to 391386b Compare November 7, 2024 09:30
@rlmanrique
Copy link
Collaborator Author

ok, completed, including the update of the collection. I tested it and works. Thanks for the idea!

Ready to be merged

@weaviate-git-bot
Copy link

Great to see you again! Thanks for the contribution.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

@rlmanrique rlmanrique force-pushed the rodrigo/delete_on_conflict branch from 391386b to 0f16511 Compare November 7, 2024 14:32
@jfrancoa jfrancoa merged commit 23a3b52 into main Nov 7, 2024
9 checks passed
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