Skip to content

Fixing MongoDB vector store support test failures #1255

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

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

sobychacko
Copy link
Contributor

Remove the requirement of MongoDB vector store auto-configuration only after MongoDataAutoConfiguration. This enables the MongoDBAtlasVectorStoreAutoConfiguration to properly provide custom Mongo conversions.

In MongoDBAtlasVectorStoreIT and MongoDbVectorStoreObservationIT tests, properly provision the MongoTemplate with custom conversions as these tests are not relying on auto-configuration.

Remove the requirement of MongoDB vector store auto-configuration only after `MongoDataAutoConfiguration`.
This enables the `MongoDBAtlasVectorStoreAutoConfiguration` to properly provide custom Mongo conversions.

In `MongoDBAtlasVectorStoreIT` and `MongoDbVectorStoreObservationIT` tests, properly provision the `MongoTemplate`
with custom conversions as these tests are not relying on auto-configuration.
@tzolov
Copy link
Contributor

tzolov commented Aug 21, 2024

This is great! Thanks @sobychacko !

@tzolov tzolov added this to the 1.0.0-M2 milestone Aug 21, 2024
@tzolov tzolov self-assigned this Aug 21, 2024
@tzolov tzolov merged commit f0771f0 into spring-projects:main Aug 21, 2024
1 check passed
@rozza
Copy link
Contributor

rozza commented Aug 21, 2024

@tzolov & @markpollack I think you can probably close out #1022.

It took an alternative approach of adding a single MimeTypeConverters class which was used by both MongoDBAtlasVectorStoreAutoConfiguration and in MongoDBAtlasVectorStoreIT. Although I think the TestApplication in #1022 is much simplier / more readable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants