Closed matheusca closed 4 years ago
Great - thanks! Looks good to me. Quick questions: Looks like we have some log statements of java.util.logging and some slf4j now - I think we should try to harmonize this - WDYT?
Perfect! I'll go to harmonize this situation ;)
Great - let me know when you are done then I can go ahead
@berndruecker done!
Thank you so much!
This PR fixes some breaking changes when trying to use apache camel 3.2.0 (it's possible use it along with apache camel 3.3.0 too)
Fix:
ScheduledBatchPollingConsumer
does not contain log instance anymore, so the consumers and producers are initiating their own log instance. Also, other classes have been changed to a constant calledLOG
, even when had their own log instance, following the standard (Apache camel release notes about this fix: https://camel.apache.org/manual/latest/camel-3x-upgrade-guide.html#_log_changed_to_private_static_log)Exchange
methods have been moved to another interface (ExtendedExchange
).camunda-bpm-camel
uses some of those methods, so some classes have been modified to useExtendedExchange
instead (Apache camel release notes about this fix: https://camel.apache.org/manual/latest/camel-3x-upgrade-guide.html#_exchange)TimePatternConvert
has been moved to other package.