camel-quarkus icon indicating copy to clipboard operation
camel-quarkus copied to clipboard
verified publisher icon apache

Remove CamelQuarkusDataFormatConfigLifecycleStrategy

Open jamesnetherton opened this issue 4 months ago • 5 comments

It should be redundant with Camel >= 4.15.0:

https://issues.apache.org/jira/browse/CAMEL-22352

CamelDataFormatRuntimeConfig can also be removed.

jamesnetherton avatar Aug 18 '25 14:08 jamesnetherton

Hi @jamesnetherton,

I’d like to work on this issue. From what I understand, since Camel >= 4.15.0 already provides the necessary changes (CAMEL-22352), we can safely remove CamelQuarkusDataFormatConfigLifecycleStrategy and also CamelDataFormatRuntimeConfig as they are now redundant.

Please let me know if I can proceed with a PR.

Thanks!

ZunedKhan07 avatar Aug 22 '25 09:08 ZunedKhan07

Please let me know if I can proceed with a PR

Unfortunately, you cannot (yet), because Camel 4.15.0 is not released yet.

jamesnetherton avatar Aug 22 '25 10:08 jamesnetherton

Should we include the removal in the PR upgrading to Camel 4.15? https://github.com/apache/camel-quarkus/pull/7808/ or maybe it requires to be deprecated first?

apupier avatar Oct 07 '25 07:10 apupier

Should we include the removal in the PR upgrading to Camel 4.15? #7808 or maybe it requires to be deprecated first?

I'll take care of it after #7808 is merged.

jamesnetherton avatar Oct 07 '25 07:10 jamesnetherton

First try at removing the ConfigLifecycleStrategy and beanio tests fail.

IllegalArgumentException: Stream name not configured. must be specified

Even though there's:

camel.dataformat.beanio.stream-name

It'll need digging into to understand what's going on.

jamesnetherton avatar Oct 08 '25 11:10 jamesnetherton