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.

Sign up for GitHub

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

Jump to bottom

Remove bytecode transformation workaround in aws2-kinesis #6238

Open
JiriOndrusek opened this issue Jun 28, 2024 · 1 comment
Open

Remove bytecode transformation workaround in aws2-kinesis #6238

JiriOndrusek opened this issue Jun 28, 2024 · 1 comment
Assignees
@JiriOndrusek
Labels
area/aws2-kinesis area/housekeeping area/native housekeeping native

Comments

@JiriOndrusek
Copy link
Contributor

Description

With the newest amazon-kinesis-client, the javax.xml.bind.DatatypeConverter is used in the code. (see ticket).

The current solution is a (long) substitution for the native to work (javax has to be replaced with jakarta).
It might (or not) be possible to solve the issue in more elegant way - by using bytecode transformations (see an example)
@JiriOndrusek JiriOndrusek self-assigned this Jun 28, 2024
@jamesnetherton jamesnetherton changed the title [camel-main] Try to use bytecode transformation instead of a substitution in aws2-kinesis Try to use bytecode transformation instead of a substitution in aws2-kinesis Jul 22, 2024
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jul 23, 2024
@jamesnetherton
Replace aws2-kinesis native substitutions with bytecode transformation …
660bc97 
…apache#6238
jamesnetherton added a commit that referenced this issue Jul 23, 2024
@jamesnetherton
Replace aws2-kinesis native substitutions with bytecode transformation …
f3a7991 
…#6238
@jamesnetherton jamesnetherton changed the title Try to use bytecode transformation instead of a substitution in aws2-kinesis Remove bytecode transformation workaround in aws2-kinesis Jul 23, 2024
@jamesnetherton
Copy link
Contributor

I did this in #6286. We just need to watch the linked ticket for a fix and eventually remove the workaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JiriOndrusek JiriOndrusek

Labels
area/aws2-kinesis area/housekeeping area/native housekeeping native
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jamesnetherton @JiriOndrusek