valdar commented on pull request #1052:
URL: 
https://github.com/apache/camel-kafka-connector/pull/1052#issuecomment-784296410


   I understand the point you are making @omarsmak , still it feels not totally 
right to me to "hardcode" such behaviour for all components (present and future 
one) by placing it in `core`. A configurable approach is a decent trade off. 
Another option would be to have a `typeconverter` (in fact you could just use 
the one in `camel-debezium-common`) that can be enabled/disabled as needed, in 
such way, if the needs manifest, we could add other `typeconverters` to handle 
from struct to something else scenarios.
   
   Moreover there is an ongoing effort to use kamelets in ckc, it is not a 
trivial change that in my opinion is best to do in steps. Once all the steps 
are completed we could much more easily change things "per connector" so all of 
this would make much less sense. In the meantime each addiction to `core` makes 
the first step toward kamelets usage in ckc little bit more difficult.... 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to