search

ancashoria / graphql-kafka-subscriptions

Apollo graphql subscriptions over Kafka protocol
MIT License
187 stars 55 forks source link

Only supports for multiple brokers #42

Closed cramhead closed 2 years ago

cramhead commented 3 years ago

Would you be willing to accept a PR for adding support for multiple brokers. I'm thinking a comma delimited list, as node-rdkafka normally uses

smilovanovic commented 2 years ago

@cramhead you should be able to do

return new KafkaPubSub({
  port: undefined,
  topic: configService.get('KAFKA_SUBSCRIPTION_TOPIC_NAME'),
  host: configService.get('KAFKA_BROKERS'),
});
cramhead commented 2 years ago

Thanks, @smilovanovic. Closing this ticket