-
Notifications
You must be signed in to change notification settings - Fork 542
ASB Concurrent Session FIFO #4092
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Joni Collinge <[email protected]>
b860249 to
6f98580
Compare
Signed-off-by: Joni Collinge <[email protected]>
|
Smoke test by running a dapr app with: Which demonstrated multiple sessions being concurrently processed whilst maintaining strict FIFO order and synchrnous processing for each session.
This means If |
Signed-off-by: Joni Collinge <[email protected]>
Signed-off-by: Joni Collinge <[email protected]>
Signed-off-by: Joni Collinge <[email protected]>
|
/ok-to-test |
Complete Build MatrixThe build status is currently not updated here. Please visit the action run below directly. Commit ref: 298ae26 |
Components certification testCommit ref: 298ae26 ❌ Some certification tests failedThese tests failed:
Additionally, some tests did not report a status:
|
Components conformance testCommit ref: 298ae26 ❌ Some conformance tests failedThese tests failed:
Additionally, some tests did not report a status:
|
Signed-off-by: Joni Collinge <[email protected]>
Signed-off-by: Joni Collinge <[email protected]>
|
/ok-to-test |
Complete Build MatrixThe build status is currently not updated here. Please visit the action run below directly. Commit ref: de6d78a |
Components certification testCommit ref: de6d78a ❌ Some certification tests failedThese tests failed:
Additionally, some tests did not report a status:
|
Components conformance testCommit ref: de6d78a ❌ Some conformance tests failedThese tests failed:
Additionally, some tests did not report a status:
|
Signed-off-by: Joni Collinge <[email protected]>
|
/ok-to-test |
Complete Build MatrixThe build status is currently not updated here. Please visit the action run below directly. Commit ref: ac25ba6 |
Components certification testCommit ref: ac25ba6 ❌ Some certification tests failedThese tests failed:
Additionally, some tests did not report a status:
|
Components conformance testCommit ref: ac25ba6 ❌ Some conformance tests failedThese tests failed:
Additionally, some tests did not report a status:
|
|
looks like the azure service bus topics certificate test passed? https://github.com/dapr/components-contrib/actions/runs/19430412140/job/55587725948 |
Description
Fixes the concurrency handling of Azure Service Bus sessions to respect FIFO ordering.
Issue reference
Please reference the issue this PR will close: #3868
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:
Note: We expect contributors to open a corresponding documentation PR in the dapr/docs repository. As the implementer, you are the best person to document your work! Implementation PRs will not be merged until the documentation PR is opened and ready for review.