davsclaus commented on PR #13038: URL: https://github.com/apache/camel/pull/13038#issuecomment-1931912932
I am against EIPs have different config classes (the only one that does that is resequencer and its a bad EIP and we should fix this in the future). It makes everything ugly and complex in DSL and tooling when this happens. component and endpoint also have all the options in one class etc. And users select what they configure. I would suggest to - mode option (configure if X or Y) can be an enum, - or you can also just add a boolean option that must be set=true to use the new behavior (but finding a good name can be tricky) - add back the old timePeriodMillis option that was removed. -- 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. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
