Skip to content

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
@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