Skip to content

Conversation

@benjamin99
Copy link
Collaborator

Summary

  • Add ReaderPrefix field to PulsarReaderSource struct for configuring SubscriptionRolePrefix
  • Update tests to cover both empty prefix (backward compatible) and custom prefix scenarios
  • Provides parity with the existing ConsumerName feature in PulsarConsumerSource

Test plan

  • Updated TestPulsarReaderSource to accept readerPrefix parameter
  • Test case 1: Empty prefix (backward compatible behavior)
  • Test case 2: Custom prefix "test-prefix" (new feature)

🤖 Generated with Claude Code

@benjamin99 benjamin99 force-pushed the feat/support-different-name-for-pulsar-reader-source branch from 871bb1d to ed9f63e Compare December 9, 2025 09:34
Add ReaderPrefix field to PulsarReaderSource that allows configuring
the SubscriptionRolePrefix when creating a Pulsar reader. This provides
parity with the ConsumerName feature in PulsarConsumerSource.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@benjamin99 benjamin99 force-pushed the feat/support-different-name-for-pulsar-reader-source branch from ed9f63e to c920465 Compare December 9, 2025 09:42
@benjamin99 benjamin99 merged commit 361ee28 into master Dec 15, 2025
0 of 7 checks passed
@benjamin99 benjamin99 deleted the feat/support-different-name-for-pulsar-reader-source branch December 15, 2025 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants