Skip to content

Conversation

@ilayaperumalg
Copy link
Member

  • Fix the QdrantValueFactory to handle "java.util.List" type
  • Update the integration test to use List type values

Resolves #3164

 - Fix the QdrantValueFactory to handle "java.util.List" type
 - Update the integration test to use List type values

Resolves spring-projects#3164

Signed-off-by: Ilayaperumal Gopinathan <[email protected]>
@ilayaperumalg ilayaperumalg added this to the 1.0.0 milestone May 15, 2025
Copy link
Member

@habuma habuma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ilayaperumalg Thanks. I just pulled down your changes, built a snapshot locally, and tried it with my small-scale example and it works. I'll try to find a time to try it with my bigger example later today, but at least at this point it looks good.

@ilayaperumalg
Copy link
Member Author

@habuma Thanks for the help testing the changes!

@sobychacko sobychacko merged commit 48fb663 into spring-projects:main May 15, 2025
2 checks passed
namsoo2 pushed a commit to namsoo2/spring-ai that referenced this pull request Jun 9, 2025
- Fix the QdrantValueFactory to handle "java.util.List" type
 - Update the integration test to use List type values

Resolves spring-projects#3164

Signed-off-by: Ilayaperumal Gopinathan <[email protected]>
Signed-off-by: minsoo.nam <[email protected]>
chedim pushed a commit to couchbaselabs/spring-ai that referenced this pull request Sep 19, 2025
- Fix the QdrantValueFactory to handle "java.util.List" type
 - Update the integration test to use List type values

Resolves spring-projects#3164

Signed-off-by: Ilayaperumal Gopinathan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unsupported Qdrant value type when using Qdrant with VectorStoreChatMemoryAdvisor

3 participants