New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Provide unshaded pulsar-kafka-client #1566
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@sijie We should probably update the docs in |
@merlimat update the doc to include section how to use |
<version>{{ site.current_version }}</version> | ||
</dependency> | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add explanation on which class to use. Eg:
If using this dependency, application would just need to change the Producer
class from org.apache.kafka.clients.producer.KafkaProducer
to org.apache.kafka.clients.producer.PulsarKafkaProducer
. Similarly, the Pulsar wrapper implementation for consumer is namedPulsarKafkaConsumer
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done updated the doc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Similar as the original pulsar-client. It is good to provide unshaded pulsar-kafka-client, so when migrating from kafka to pulsar, there would be a change that original kafka client will be colocated with the pulsar-kafka-client. Providing an unshaded pulsar-kafka-client will address this concern.