morganstanley/modern-cpp-kafka

Producer Config should include Properties

andrewchellis opened this issue · 0 comments

Hi, I am seeing compiler errors when including kafka/ProducerConfig.h as it cannot resolve the Properties class for

class ProducerConfig : public Properties {

Similar to kafka/ConsumerConfig.h and kafka/AdminClientConfig.h, ProducerConfig.h should probably not include kafka/Types.h and should instead include kafka/Properties.h