gnodet commented on PR #22086:
URL: https://github.com/apache/camel/pull/22086#issuecomment-4087938770

   @jamesnetherton You're right, I was wrong. The encoding format change was a 
red herring — the SDK does handle base64→float conversion via 
`EmbeddingValue.asFloats()`.
   
   The real issue is that when no embedding model is specified (e.g. running 
with local Ollama without setting the `EMBEDDING_MODEL` property), the producer 
sets an exception on the exchange at line 70, but the tests never check 
`exchange.getException()`. They proceed to cast the body (still the original 
input String) to `List<Float>`, causing the ClassCastException.
   
   I'll rework this PR with the proper fix. Sorry for the incorrect analysis.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to