Skip to content

Revise backoff implementation for Async weaviate knowledgebase. #27

Description

@jacobthebanana

Describe the bug

Backoff isn't handling a certain number of exceptions in the Async Weaviate knowledge base tool implementation.

To Reproduce

Run with concurrency of e.g., 36 and see the exceptions show up.

Expected behavior

Exponential backoff should be activated when any weaviate connectivity error happens.

Screenshots

If applicable, add screenshots to help explain your problem.

Version

  • Version info such as v0.1.5

Additional context

Possible fix- instead of relying on backoff which allows only built-in exceptions, do the exception handling ourselves and throw e.g., a classic Runtime error which we handle using backoff.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions