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

Improve Weaviate error handling #4233

Closed
vga91 opened this issue Nov 19, 2024 · 0 comments · Fixed by #4273 or #4264
Closed

Improve Weaviate error handling #4233

vga91 opened this issue Nov 19, 2024 · 0 comments · Fixed by #4273 or #4264

Comments

@vga91
Copy link
Collaborator

vga91 commented Nov 19, 2024

If we create a collection 'TestCollection' e.g. with size 4

CALL apoc.vectordb.weaviate.query($host, 'TestCollection', [0.2, 0.1, 0.9], null, 5, $conf)

And we execute a weaviate query with an embedding with size e.g. 3, so that it doesn't match the one specified with the schema collection,
it will returned an unclear error size:

CALL apoc.vectordb.weaviate.query($host, 'TestCollection', [0.2, 0.1, 0.9], null, 5, $conf)

The result is:

java.lang.NullPointerException: Cannot invoke "java.util.List.stream()" because "collectionValue" is null

Instead, it should return the error cause.

@vga91 vga91 moved this to In Progress in APOC Extended Larus Nov 19, 2024
RobertoSannino pushed a commit that referenced this issue Dec 2, 2024
* Fixes #4233: Improve Weaviate error handling

* fix tests
@vga91 vga91 closed this as completed Dec 4, 2024
@github-project-automation github-project-automation bot moved this from Review to Done (to cherry-pick) in APOC Extended Larus Dec 4, 2024
vga91 added a commit that referenced this issue Dec 5, 2024
* Fixes #4233: Improve Weaviate error handling

* fix tests
vga91 added a commit that referenced this issue Dec 5, 2024
* Fixes #4233: Improve Weaviate error handling

* fix tests
vga91 added a commit that referenced this issue Dec 5, 2024
* Fixes #4233: Improve Weaviate error handling

* fix tests
vga91 added a commit that referenced this issue Dec 5, 2024
* Fixes #4233: Improve Weaviate error handling

* fix tests
@vga91 vga91 moved this from Done (check if cherry-pick) to Done in APOC Extended Larus Dec 5, 2024
vga91 added a commit that referenced this issue Dec 10, 2024
* Fixes #4233: Improve Weaviate error handling

* fix tests
vga91 added a commit that referenced this issue Dec 10, 2024
* Fixes #4233: Improve Weaviate error handling

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