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

Multithreaded support for Eventhubs management family of APIs #5315

Merged
Merged
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
41a231d
Enabled multithreaded calls to eventhubs management APIs
LarryOsterman Feb 2, 2024
ab5f7fa
Merge branch 'main' into larryo/multithreaded_management
LarryOsterman Feb 2, 2024
14ac359
Added partition client properties tests, removed unused variables in …
LarryOsterman Feb 2, 2024
72edbb8
Restructured properties APIs in eventhubs to simplify producer client…
LarryOsterman Feb 2, 2024
40dfecb
Clang fix
LarryOsterman Feb 2, 2024
1b7f2a6
Removed AMQP code which logged an incoming management message
LarryOsterman Feb 2, 2024
d1a41e7
Removed unused lambda capture fields.
LarryOsterman Feb 3, 2024
84a5429
Fixed test crash in LinkAttachDetach AMQP test
LarryOsterman Feb 5, 2024
8a4c96e
Added test cases for management authn failures
LarryOsterman Feb 5, 2024
2ac800f
clang-format
LarryOsterman Feb 5, 2024
99cc884
Don't emit body contents in AmqpMessage insertion operator
LarryOsterman Feb 6, 2024
9089926
Don't take numeric value parameters by value to ostream insertion ope…
LarryOsterman Feb 6, 2024
847296a
clang-format
LarryOsterman Feb 6, 2024
c90ca72
Merge branch 'main' into larryo/multithreaded_management
LarryOsterman Feb 6, 2024
924744c
Merge branch 'main' into larryo/multithreaded_management
LarryOsterman Feb 6, 2024
b31ccd9
Updated changelog to reflect changes in this PR
LarryOsterman Feb 6, 2024
64cdeb1
Updated eventhubs dependency to match reality
LarryOsterman Feb 7, 2024
1307c98
Pull request feedback
LarryOsterman Feb 8, 2024
5964ef9
Fixed test crashes in management tests
LarryOsterman Feb 8, 2024
51e432b
clang-format
LarryOsterman Feb 8, 2024
db550b9
Improved code coverage
LarryOsterman Feb 8, 2024
198af85
compiler didnt notice an impossible branch
LarryOsterman Feb 8, 2024
60318a3
clang-format
LarryOsterman Feb 8, 2024
a373ae5
Better code coverage
LarryOsterman Feb 8, 2024
29a9c47
clang fixes
LarryOsterman Feb 8, 2024
5209f1f
amqpvalue_create_described does not clone its inputs
LarryOsterman Feb 9, 2024
395e1c3
Added value based tests for enumeration stream inserters
LarryOsterman Feb 9, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
clang-format
LarryOsterman committed Feb 6, 2024
commit 847296aae7e0920ff5045e4f9ff5029be1535109
Original file line number Diff line number Diff line change
@@ -78,10 +78,10 @@ namespace Azure { namespace Core { namespace Amqp { namespace Models {

/**
* @brief ostream insertion operator for AmqpValueType.
*
*
* @param os - stream to insert to.
* @param value - value to insert.
*
*
* @returns the input ostream.
*/
std::ostream& operator<<(std::ostream& os, AmqpValueType const value);
LarryOsterman marked this conversation as resolved.
Show resolved Hide resolved