search

apache / camel-quarkus

Apache Camel Quarkus
https://camel.apache.org
Apache License 2.0
255 stars 189 forks source link

Reconsider usage of `quarkus-tika` in `camel-quarkus-tika` extension #5393

Open jamesnetherton opened 1 year ago

jamesnetherton commented 1 year ago

Description

From my POV, usage of quarkus-tika is causing a bit of maintenance pain. Using it means we often get out of sync with Camel for tika, pdfbox & fontbox. Resulting in having to disable tests or leave the native builds for some extensions broken. See https://github.com/apache/camel-quarkus/issues/5234 & https://github.com/apache/camel-quarkus/issues/5244.

quarkus-tika also imposes some limitations on us in terms of functionality:

https://camel.apache.org/camel-quarkus/next/reference/extensions/tika.html#extensions-tika-camel-quarkus-limitations

It may be better to copy the native support to our tika extension.

b-thiswatch commented 10 months ago

@JiriOndrusek I was just about to open a new issue, since I experienced this problem in:

https://github.com/quarkiverse/quarkus-tika/issues/176

I also just build this reproducer https://github.com/thiswatch/quarkus-tika-camel-reproducer which you probably then don't need anymore, right?

What's the best approach?

JiriOndrusek commented 10 months ago

Hi @b-thiswatch , I'd say that this ticket should be enough to ask for the fix. I haven't had any time to look into the problem so far. If you would like to work on a solution, we can cooperate on the PR.

b-thiswatch commented 10 months ago

I'd love to help, but given how long it took me to even figure out what is wrong, I might not be of great help :-/