- Issue created by @scott_euser
- Status changed to Active
7 months ago 10:06pm 23 November 2023 - 🇬🇧United Kingdom scott_euser
Now that we have another vector client, I can see this is a problem. Updated description.
- Assigned to scott_euser
- 🇬🇧United Kingdom scott_euser
Need to test this from start to finish with Pinecone and Milvus again, but initial checks working fine. Throwing extra exceptions in the case that keys in $parameters array are missing when required to support developers using these vector client plugins.
- Status changed to Needs review
7 months ago 2:46pm 25 November 2023 - Status changed to Needs work
7 months ago 2:53pm 25 November 2023 - 🇬🇧United Kingdom scott_euser
Need to review further wherever $vector_client->*() is called to switch to use $parameters instead. Eg in the .module file, search test controller, and stats page.
- Merge request !66Resolve #3400915 "Update vectorclientinterface to" → (Merged) created by scott_euser
-
scott_euser →
committed c5feead5 on 1.0.x
Issue #3400915 by scott_euser: Update VectorClientInterface to allow for...
-
scott_euser →
committed c5feead5 on 1.0.x
- Issue was unassigned.
- Status changed to Fixed
7 months ago 7:44am 26 November 2023 - 🇬🇧United Kingdom scott_euser
Done full test through Pinecone and Milvus again.
Automatically closed - issue fixed for 2 weeks with no activity.