[GitHub] [camel-quarkus] zhfeng commented on issue #4525: nitrite test fails to compile to native with Camel 4 and Quarkus 3

["zhfeng (via GitHub)" <gi...@apache.org>]

zhfeng commented on issue #4525:
URL: https://github.com/apache/camel-quarkus/issues/4525#issuecomment-1425128698

   Hmm, the transitive dependency `uk.co.jemos.podam:podam` only supports Java EE 7 and I doubt they have no plan to upgrade, see https://github.com/mtedone/podam/issues/311
   
   Our tests work in JVM mode because the `@Id` field class  is primitive. see https://github.com/apache/camel-quarkus/blob/d11a8945994ad16a5414fc1b6ff491e138ed7cfe/integration-tests/nitrite/src/main/java/org/apache/camel/quarkus/component/nitrite/it/EmployeeMappable.java#L33-L37
   
   So there is no need to call `dummyDocument` in 
   https://github.com/nitrite/nitrite-java/blob/3.x/nitrite/src/main/java/org/dizitart/no2/util/ValidationUtils.java#L210
   
   But during native building, it needs to static analyst all the branches. I belive we can catch the same issue in JVM mode if we use a Java class as an `@Id` filed.
   
   Anyway, I can try to add a subsitution on `DocumentUtils.dummyDocument` to throw an UnSupported exception. But there is still a limitation on `camel-nitrite`, I think we need to add some notes in camel docs @davsclaus ?
   
   And it seems that `nitrite-4.x` will move `uk.co.jemos.podam:podam` to test dependency but there is no any release for 4.x. 


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org