David Kjerrumgaard
David Kjerrumgaard
@eolivelli , Can you please take a look at this when you get the chance? Thanks again!
@eolivelli Can I please get some feedback on these changes I made in response to your initial feedback? Thanks again for the review, I really appreciated it.
/pulsarbot run-failure-checks
@tisonkun @eolivelli I would appreciate another review when you have the time.
Can you include the documentation changes for this in the PR as well?
Missing Features: ====================== Schema definition, management. View AVRO encoded messages. Drop messages into topics. Lenses SQL (KSQL under the covers)
Any update on this?
FWIW, when I reverted the [change](https://github.com/langchain4j/langchain4j-embeddings/pull/26), the issue goes away, and I am able to use the AllMiniLmL6V2EmbeddingModel() class inside my Pulsar Function.
Client Version 4.0.2
It looks like the PR has one commit/change request and some conflicts. Until those are resolved, we cannot merge the PR.