quarkiverse / quarkus-langchain4j

Quarkus Langchain4j extension
https://docs.quarkiverse.io/quarkus-langchain4j/dev/index.html
Apache License 2.0
119 stars 65 forks source link

Add a warning about missing -parameters flag for javac #713

Closed geoand closed 3 days ago

geoand commented 3 days ago

Without this warning, users can end up with only an error complaining about a duplicate null key

This was prompted by https://stackoverflow.com/q/78695273/2504224, but it's far from the first time I've seen the same issue.

geoand commented 3 days ago

@jmartisk if you don't have time to look at the Neo4j failures, let me know please so I can disable them

jmartisk commented 3 days ago

@geoand Hmmm supercool, another problematic embedding store that just stops working? I'll look into it asap, but feel free to disable for now

geoand commented 3 days ago

but feel free to disable for now

I'll wait a little in case you come up with an easy fix :)

jmartisk commented 3 days ago

Ok I can't even reproduce it locally, this will be tricky

jmartisk commented 3 days ago

I think it's something with neo4j, because with neo4j:5.20.0 image it works, with the recently released 5.21.0 it fails