Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@cbullinger
Copy link
Collaborator

@cbullinger cbullinger commented Nov 3, 2025

Adds vector-search endpoint: GET /api/movies/vector-search - Search movies using MongoDB Vector Search with VoyageAI embeddings

  1. Add VOYAGE_API_KEY to your .env
  2. Build and run:
mvn clean install
mvn spring-boot:run

The application will start on http://localhost:3001

  1. Test out endpoint on http://localhost:3001/swagger-ui.html

@cbullinger cbullinger marked this pull request as ready for review November 4, 2025 15:56
Copy link
Collaborator

@shuangela shuangela left a comment

Choose a reason for hiding this comment

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

lgtm % a few nits

* @throws IOException If there's an error calling the API
* @throws InterruptedException If the HTTP request is interrupted
*/
private List<Double> generateVoyageEmbedding(String text, String apiKey)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm curious why you used this HTTP request strategy vs the code we have in the vector search docs: https://www.mongodb.com/docs/atlas/atlas-vector-search/create-embeddings/?embedding-model=voyage&data-source=existing&language-no-interface=java-sync#define-a-method-to-generate-vector-embeddings.-1. Both work, but I'm curious for my own knowledge!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

because Augment really really wanted to keep reverting back to HTTP requests 😓 thanks for calling this out. I've updated to reflect the docs' strategy

@cbullinger cbullinger merged commit bffca50 into development Nov 5, 2025
1 check passed
@cbullinger cbullinger deleted the docsp-54427-vector branch November 5, 2025 01:09
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