Closed eduardopoleoflipp closed 3 years ago
@eduardopoleoflipp looks good - please add a line in the CHANGELOG under UNRELEASED and I can merge it in!
hey @dorner, I fixed the conflict in the CHANGELOG I think this should be good to go now!
Merged - thanks very much!
Pull Request Template
Description
The
kafka_producer
interface is outdated at this point and it's only there for backwards compatibility. We're moving all the validation checks into thekafka_producers
and throwing a deprecation warning if the old interface is used.Incidentally, this also takes care of the infinite loop we were seeing before when no producer method was defined.
Fixes # (https://github.com/flipp-oss/deimos/issues/90)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: