[ 
https://issues.apache.org/jira/browse/CAMEL-16438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17312428#comment-17312428
 ] 

Claus Ibsen commented on CAMEL-16438:
-------------------------------------

Yes I removed all the .configuration.xxx and used just plain names, all over 
the code and config files.
The unit test passes.
And debugging the main class also shows the options are set on the component 
class via its configurer that knows which setter to use.

Yes you can test yourself and also use 3.9.0 now that its released.


> Kafka component properties randomly end up under camel.component.kafka or 
> camel.component.kafka.configuration
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-16438
>                 URL: https://issues.apache.org/jira/browse/CAMEL-16438
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-kafka, camel-spring-boot
>    Affects Versions: 3.8.0, 3.9.0
>            Reporter: David Oftedal
>            Priority: Minor
>             Fix For: 3.10.0
>
>         Attachments: kafka-chaos.zip
>
>
> According to 
> [https://camel.apache.org/manual/latest/camel-3x-upgrade-guide-3_2.html#_configuring_components_via_spring_boot_auto_configuration]
>  , Camel component configuration should as of version 3.2 no longer use the 
> .configuration prefix.
> In version 3.8.0, many properties have indeed moved out of .configuration. 
> But some properties are still only set if set with the .configuration prefix. 
> The same seems to be the case in 3.9.0 as well.
>  As far as I can tell, this non-exhaustive list contains valid examples:
>  * camel.component.kafka.configuration.brokers (specifically mentioned in the 
> documentation as having moved)
>  * camel.component.kafka.configuration.max-poll-records
>  * camel.component.kafka.configuration.group-id
>  * camel.component.kafka.configuration.topic
>  * camel.component.kafka.configuration.ssl-keystore-password
>  * camel.component.kafka.configuration.ssl-truststore-password
>  * camel.component.kafka.configuration.value-deserializer
> Some properties seem to have moved *to* .configuration:
>  * camel.component.kafka.configuration.allow-manual-commit
>  * camel.component.kafka.configuration.break-on-first-error
> Some related properties now appear to be split between two prefixes:
>  * camel.component.kafka.ssl-keystore-location and 
> camel.component.kafka.configuration.ssl-keystore-password
>  * camel.component.kafka.ssl-truststore-location and 
> camel.component.kafka.configuration.ssl-truststore-password
>  * camel.component.kafka.value-serializer and 
> camel.component.kafka.configuration.value-deserializer
> Trying to figure out what goes where requires a pretty exhaustive and 
> error-prone search.
>  Obviously it's possible to copy and paste each property so that it's set 
> both with and without .configuration, but if the intent is to remove 
> .configuration altogether, would it be possible to either confirm or negate 
> what we're seeing?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to