Skip to content

Develop to Main#46

Merged
sujata-m merged 2 commits intomainfrom
develop
Mar 13, 2026
Merged

Develop to Main#46
sujata-m merged 2 commits intomainfrom
develop

Conversation

@sujata-m
Copy link
Contributor

Introduce an optional max_receivable_messages parameter to TopicAbstract.subscribe and implement support in AzureTopic and LocalTopic. AzureTopic now tracks per-receiver local_received_messages, limits receive batch sizes accordingly, and stops once the requested max is reached. Update example to use 'test-response', disable the random exception, fix task sleep to 5s, and lower concurrency to 1 with subscribe(..., max_receivable_messages=1). Bump package version to 0.0.24.

Testing done:

  • Invoked example.py with a test topic published 10 messages
  • Tested with Max concurrency 2 and Max messages 5 - Working as expected
  • Tested with max concurrency 5 and max messages 2 - Working as expected
  • Tested with max concurrency 2 and max messages 0 - Working as expected. Stays listening forever
  • Tested with max concurrency 2 and no input for max messages - working as expected. Listens forever

Caveat

  • This is not tested or implemented in local_topic.py. This does not work with Local provider

susrisha added 2 commits March 5, 2026 11:07
Introduce an optional max_receivable_messages parameter to TopicAbstract.subscribe and implement support in AzureTopic and LocalTopic. AzureTopic now tracks per-receiver local_received_messages, limits receive batch sizes accordingly, and stops once the requested max is reached. Update example to use 'test-response', disable the random exception, fix task sleep to 5s, and lower concurrency to 1 with subscribe(..., max_receivable_messages=1). Bump package version to 0.0.24.
…sages

Add max_receivable_messages to topic subscribe
@sujata-m sujata-m requested review from MashB and susrisha March 13, 2026 08:43
@sujata-m sujata-m merged commit 9feffa4 into main Mar 13, 2026
7 checks passed
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.

3 participants