Skip to content
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

Event Hubs: block processor on stop and run live tests on PRs #41878

Merged
merged 11 commits into from
Sep 18, 2024

Conversation

lmolkova
Copy link
Member

@lmolkova lmolkova commented Sep 17, 2024

  • Changes EventProcessorClient.stop() to block for up to 10sec to wait for processor to close
  • Adds EventProcessorClient.stop(Duration) to block for specific timeout - it throws if timeout occurs
  • triggers live tests from PR
  • requires live tests for release

@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

com.azure:azure-messaging-eventhubs

@lmolkova lmolkova changed the title Event Hubs: block processor on stop and run live tests in CI pipeline Event Hubs: block processor on stop and run live tests on PRs Sep 17, 2024
@lmolkova lmolkova closed this Sep 17, 2024
@lmolkova lmolkova reopened this Sep 17, 2024
@lmolkova lmolkova force-pushed the eventhubs-close-processor-and-live-tests branch from fec782c to 167252f Compare September 17, 2024 01:43
@lmolkova lmolkova closed this Sep 17, 2024
@lmolkova lmolkova reopened this Sep 17, 2024
@lmolkova lmolkova closed this Sep 17, 2024
@lmolkova lmolkova reopened this Sep 17, 2024
@lmolkova lmolkova force-pushed the eventhubs-close-processor-and-live-tests branch from f11ed76 to 032e7d1 Compare September 17, 2024 21:41
@lmolkova lmolkova merged commit cd54147 into Azure:main Sep 18, 2024
32 checks passed
mssfang pushed a commit to mssfang/azure-sdk-for-java that referenced this pull request Oct 21, 2024
…41878)

* EH processor: block on stop and run live tests in CI pipeline
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants