-
Notifications
You must be signed in to change notification settings - Fork 55
Open
Description
Hi,
we encountered issue with your package, when private method createConsumer in kafka-pubsub.js does not handle node-rdkafka Stream API in correct way. This problem leads to not catching all messages from Kafka broker, in our case, after 3 events were recieved, consumer stops consuming.
Current code:
stream.consumer.on('data', (message) => {
let parsedMessage = JSON.parse(message.value.toString())
...
})You access Standard API via Stream API object without manual setup for flowing/non-flowing mode (see examples on https://www.npmjs.com/package/node-rdkafka). One of the solutions is use only Stream API (example below).
Fix:
stream.on('data', (message) => {
let parsedMessage = JSON.parse(message.value.toString())
...
})This fix helps us to get it working, so we'll be glad to have this thing fixed in the next release. Thanks!
vahalova, tomec-martin, cervajs, opravil-jan, kk616 and 4 morekk616 and cubase
Metadata
Metadata
Assignees
Labels
No labels