Skip to content

feat: streaming test app #144

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 22 commits into from
Apr 2, 2025
Merged

feat: streaming test app #144

merged 22 commits into from
Apr 2, 2025

Conversation

micpapal
Copy link
Member

@micpapal micpapal commented Apr 2, 2025 •
edited
Loading

Description

Please provide a meaningful description of what this change will do, or is for.
Bonus points for including links to related issues, other PRs, or technical
references.

Note that by not including a description, you are asking reviewers to do extra
work to understand the context of this change, which may lead to your PR taking
much longer to review, or result in it not being reviewed at all.

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

msardara and others added 20 commits March 26, 2025 20:13
@msardara
chore(service): simplify session trait with async_trait
48340a2 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
feat: request reply sessions
5aa97c0 
Signed-off-by: Mauro Sardara <[email protected]>
@micpapal
fix: fix get_source util usage
854451a 
Signed-off-by: Michele Papalini <[email protected]>
@msardara
feat: request reply sessions
ce0609e 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
feat: request reply sessions
a6fe814 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
2a3e231 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
69726bb 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
295e084 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
00365f0 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
234443e 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
Merge branch 'main' into refactor/utils
42997d4 
* main:
  feat: streaming session type (#132)
  feat: request/reply session type (#124)
@micpapal
feat: streaming in test app
e44921a 
Signed-off-by: Michele Papalini <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
40d5187 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
03bd5da 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
9158f6b 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
74b0dff 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
fefa687 
Signed-off-by: Mauro Sardara <[email protected]>
@msardara
refactor: improve utils classes and simplify message processor
46d26b2 
Signed-off-by: Mauro Sardara <[email protected]>
@micpapal
Merge branch 'refactor/utils' into feat/streaming-test-app
d8541bd 
Signed-off-by: Michele Papalini <[email protected]>
@micpapal
fix: fix producer app
f1c1cde 
Signed-off-by: Michele Papalini <[email protected]>
@micpapal micpapal changed the title Feat/streaming test app Feat: streaming test app Apr 2, 2025
@micpapal micpapal changed the title Feat: streaming test app feat: streaming test app Apr 2, 2025
@micpapal micpapal force-pushed the feat/streaming-test-app branch from 0d56ded to 7dbd212 Compare April 2, 2025 13:57
@micpapal micpapal marked this pull request as ready for review April 2, 2025 14:01
@micpapal micpapal requested a review from a team April 2, 2025 14:01
msardara
msardara approved these changes Apr 2, 2025
View reviewed changes
@micpapal
fix: fix unit tests in streaming
aaa8ac1 
Signed-off-by: Michele Papalini <[email protected]>
@micpapal micpapal merged commit 59b6dea into main Apr 2, 2025
17 checks passed
@micpapal micpapal deleted the feat/streaming-test-app branch April 2, 2025 15:08
@build-agntcy build-agntcy mentioned this pull request Apr 2, 2025
Closed
msardara added a commit that referenced this pull request Apr 3, 2025
@msardara
Merge branch 'main' into test/req-reply
6d96408 
* main:
  refactor: remove locks in streaming session layer (#145)
  feat: streaming test app (#144)
  fix: windows error when compiling aws-lc-rs (#143)
This was referenced Apr 4, 2025
Merged
Merged
This was referenced Jun 2, 2025
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@msardara msardara msardara approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@micpapal @msardara