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

The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present. #4231

Closed
vga91 opened this issue Nov 18, 2024 · 0 comments · May be fixed by #4264
Closed

The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present. #4231

vga91 opened this issue Nov 18, 2024 · 0 comments · May be fixed by #4264
Labels
extended-functionality

Comments

@vga91
Copy link
Contributor

vga91 commented Nov 18, 2024

The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present.

Expected Behavior (Mandatory)

It should get the fields config from mapping.mappingKey config

Actual Behavior (Mandatory)

It fails with error:

You have to define `field` list of parameter to be returned

How to Reproduce the Problem

Execute:

CALL apoc.vectordb.milvus.queryAndUpdate($host, 'test_collection', [0.2, 0.1, 0.9, 0.7], null, 5, 
{allResults: true, mapping: { relType: 'TEST',  entityKey: 'readID', metadataKey: 'foo' } })
@vga91 vga91 moved this to Todo in APOC Extended Larus Nov 18, 2024
vga91 pushed a commit that referenced this issue Nov 20, 2024
@gmarcostam @vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
9e90fba 
…te.query* procedures should get the fields config from metadataKey if present
vga91 added a commit that referenced this issue Nov 20, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
a4201cb 
…te.query* procedures should get the fields config from metadataKey if present
@vga91 vga91 mentioned this issue Nov 20, 2024
Merged
vga91 added a commit that referenced this issue Nov 29, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
aefcee1 
…te.query* procedures should get the fields config from metadataKey if present
vga91 added a commit that referenced this issue Dec 3, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
922f312 
…te.query* procedures should get the fields config from metadataKey if present
vga91 added a commit that referenced this issue Dec 4, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
5998d12 
…te.query* procedures should get the fields config from metadataKey if present
RobertoSannino pushed a commit that referenced this issue Dec 10, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
5748e1c 
…te.query* procedures should get the fields config from metadataKey if present (#4241)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
vga91 added a commit that referenced this issue Dec 10, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
895eab0 
…te.query* procedures should get the fields config from metadataKey if present (#4241)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
@vga91 vga91 mentioned this issue Dec 10, 2024
Merged
vga91 added a commit that referenced this issue Dec 11, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
637add6 
…te.query* procedures should get the fields config from metadataKey if present (#4241)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
vga91 added a commit that referenced this issue Dec 11, 2024
@vga91
Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weavia…
6f0ecc6 
…te.query* procedures should get the fields config from metadataKey if present (#4241) (#4298)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
@vga91 vga91 closed this as completed Dec 11, 2024
@github-project-automation github-project-automation bot moved this from Review to Done (check if cherry-pick) in APOC Extended Larus Dec 11, 2024
@vga91 vga91 mentioned this issue Dec 18, 2024
Draft
vga91 added a commit that referenced this issue Dec 18, 2024
@vga91
[NOID] Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb…
0b223e2 
….weaviate.query* procedures should get the fields config from metadataKey if present (#4241)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
vga91 added a commit that referenced this issue Dec 18, 2024
@vga91
[NOID] Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb…
5dc0871 
….weaviate.query* procedures should get the fields config from metadataKey if present (#4241)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
vga91 added a commit that referenced this issue Dec 18, 2024
@vga91
[NOID] Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb…
cde1151 
….weaviate.query* procedures should get the fields config from metadataKey if present (#4241)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
vga91 added a commit that referenced this issue Dec 19, 2024
@vga91
[NOID] Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb…
c2a1a90 
….weaviate.query* procedures should get the fields config from metadataKey if present (#4241)

* Fixes #4231: The apoc.vectordb.milvus.query* and apoc.vectordb.weaviate.query* procedures should get the fields config from metadataKey if present

* test fixes and changes review

* fix tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
extended-functionality
Projects
APOC Extended Larus
Status: Done (check if cherry-pick)
Milestone
No milestone
1 participant
@vga91