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.

Sign up for GitHub

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

Jump to bottom

async: update geo-index asynchronously #3707

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

async: update geo-index asynchronously #3707

wants to merge 1 commit into from

Conversation

asdine
Copy link
Member

@asdine asdine commented Oct 31, 2023 •
edited
Loading

What's being changed:

The geo-index is now updated asynchronously by the shared workers.
Fixes weaviate/weaviate-chaos-engineering#147

Review checklist

@sonarcloud
Copy link

sonarcloud bot commented Oct 31, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@asdine asdine marked this pull request as ready for review October 31, 2023 13:56
@stale Stale
Copy link

stale bot commented Mar 17, 2024

Thank you for your contribution to Weaviate. This issue has not received any activity in a while and has therefore been marked as stale. Stale issues will eventually be autoclosed. This does not mean that we are ruling out to work on this issue, but it most likely has not been prioritized high enough in the last months.
If you believe that this issue should remain open, please leave a short reply. This lets us know that the issue is not abandoned and acts as a reminder for our team to consider prioritizing this again.
Please also consider if you can make a contribution to help with the solution of this issue. If you are willing to contribute, but don't know where to start, please leave a quick message and we'll try to help you.
Thank you, The Weaviate Team

@stale stale bot added the autoclosed Closed by the bot. We still want this, but it didn't quite make the latest prioritization round label Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aliszka aliszka aliszka approved these changes

@trengrj trengrj Awaiting requested review from trengrj

Assignees
No one assigned
Labels
autoclosed Closed by the bot. We still want this, but it didn't quite make the latest prioritization round
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Regression in pipeline Preventing crashing of geo properties ... with async
2 participants
@asdine @aliszka