Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
jamesnetherton opened this issue Oct 2, 2023 · 3 comments · Fixed by #6963
Closed

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

jamesnetherton opened this issue Oct 2, 2023 · 3 comments · Fixed by #6963

Comments

@jamesnetherton
Copy link
Contributor

jamesnetherton commented Oct 2, 2023

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 #5234 & #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
Copy link

b-thiswatch commented Nov 14, 2023

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

quarkiverse/quarkus-tika#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
Copy link
Contributor

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
Copy link

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 :-/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants