Closed somratdutta closed 3 weeks ago
Hi @scottf, I have made some changes as per the latest discussion. You can see the commit here. I also agree on your PR for supporting Serializable Consumer Configuration, I would refactor the Jetstream connector code once its merged.
Upcoming improvements:
Updates in this commit :
Removed NatsConsumeOptions and updated configurations in NatsJetStreamSource and related classes:
Fixed Method Visibility in ExampleErrorListener to Match Interface Requirements:
Refactored NatsJetStreamSourceTest:
Note : This refactoring is done based on this PR which adds Serialisable wrappers. It hasn't been released yet, so the jnats version is bumped for next release.
@scottf
Hey our team is interested in this! Do we know the status of this PR?
We have added support for NATS JetStream for Source. Also added is a test-case for the Source class. We are open to feedback and actively working on to improve the current state of this project.