Avoid trying to retrieve the default ds if a named one should be used

[jmartisk]

Fixes #667

[geoand]

Thanks!

[geoand]

Any chance we can get a test?

[jmartisk]

Hmm, the tests in pgvector module are using @QuarkusTest. We will probably need to rewrite them to use quarkus-junit5-internal because they can't be mixed together.

[jmartisk]

I'll try that

[geoand]

🙏

[jmartisk]

After switching to quarkus-junit5-internal some of the existing tests start throwing really weird errors (IllegalArgumentException from reflectively calling methods with wrong arguments), I think it has something to do with being parametrized via @MethodSource (https://github.com/langchain4j/langchain4j/blob/0.31.0/langchain4j-core/src/test/java/dev/langchain4j/store/embedding/EmbeddingStoreWithFilteringIT.java#L27), maybe the Quarkus-internal testing framework doesn't support that.

[geoand]

Indeed they don't AFAIR

[jmartisk]

We can work around it by temporarily extending EmbeddingStoreIT instead of EmbeddingStoreWithFilteringIT, then we won't have any parametrized tests, but we would lose test coverage of metadata filtering...

[geoand]

Up to you